Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

mdPanel: Add intelligent positioning to the panel #7878

Closed
KarenParker opened this issue Apr 4, 2016 · 1 comment
Closed

mdPanel: Add intelligent positioning to the panel #7878

KarenParker opened this issue Apr 4, 2016 · 1 comment
Assignees
Labels
has: Pull Request A PR has been created to address this issue resolution: fixed
Milestone

Comments

@KarenParker
Copy link
Contributor

This task involves:

  • determining if it is offscreen and reposition if possible by flipping it
  • accepting a priority list of positions
    • try each position and find the first one that is positioned onscreen
    • last is a default position
@KarenParker KarenParker added in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs for: internal contributor The team will address this issue and community PRs are not requested. EOC labels Apr 4, 2016
@KarenParker KarenParker added this to the EOC - Q1 milestone Apr 4, 2016
@ThomasBurleson ThomasBurleson modified the milestones: EOC - Q1, - Backlog May 26, 2016
@ErinCoughlan
Copy link
Contributor

Update: Multiple positions are now accepted. The remaining task in this issue to force position the panel if none of the provided options work.

@ErinCoughlan ErinCoughlan removed the EOC label Sep 9, 2016
@ErinCoughlan ErinCoughlan removed the for: internal contributor The team will address this issue and community PRs are not requested. label Sep 9, 2016
@crisbeto crisbeto added the has: Pull Request A PR has been created to address this issue label Sep 20, 2016
crisbeto added a commit to crisbeto/material that referenced this issue Sep 20, 2016
Prevents the panel from going outside the viewport by adjusting the position.
If developers want more control over how the panel gets repositioned, they can specify addition fallback positions via `addPanelPosition`.

Related to angular#9641.

Fixes angular#7878.
crisbeto added a commit to crisbeto/material that referenced this issue Sep 20, 2016
Prevents the panel from going outside the viewport by adjusting the position.
If developers want more control over how the panel gets repositioned, they can specify addition fallback positions via `addPanelPosition`.

Related to angular#9641.

Fixes angular#7878.
crisbeto added a commit to crisbeto/material that referenced this issue Sep 20, 2016
Prevents the panel from going outside the viewport by adjusting the position.
If developers want more control over how the panel gets repositioned, they can specify addition fallback positions via `addPanelPosition`.

Related to angular#9641.

Fixes angular#7878.
crisbeto added a commit to crisbeto/material that referenced this issue Sep 21, 2016
Prevents the panel from going outside the viewport by adjusting the position.
If developers want more control over how the panel gets repositioned, they can specify addition fallback positions via `addPanelPosition`.

Related to angular#9641.

Fixes angular#7878.
crisbeto added a commit to crisbeto/material that referenced this issue Sep 26, 2016
Prevents the panel from going outside the viewport by adjusting the position.
If developers want more control over how the panel gets repositioned, they can specify addition fallback positions via `addPanelPosition`.

Related to angular#9641.

Fixes angular#7878.
kara pushed a commit that referenced this issue Oct 16, 2016
Prevents the panel from going outside the viewport by adjusting the position.
If developers want more control over how the panel gets repositioned, they can specify addition fallback positions via `addPanelPosition`.

Related to #9641.

Fixes #7878.
@Splaktar Splaktar removed this from the - Backlog milestone Feb 23, 2019
@Splaktar Splaktar added resolution: fixed and removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs labels Mar 7, 2020
@Splaktar Splaktar added this to the 1.1.2 milestone Mar 7, 2020
@angular angular locked as resolved and limited conversation to collaborators Mar 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has: Pull Request A PR has been created to address this issue resolution: fixed
Projects
None yet
Development

No branches or pull requests

6 participants