Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Popover] Doesn't reposition with anchorEl #5937

Closed
mbrookes opened this issue Jan 15, 2017 · 2 comments
Closed

[Popover] Doesn't reposition with anchorEl #5937

mbrookes opened this issue Jan 15, 2017 · 2 comments
Labels
bug 🐛 Something doesn't work component: Popover The React component. good first issue Great for first contributions. Enable to learn the contribution process.

Comments

@mbrookes
Copy link
Member

mbrookes commented Jan 15, 2017 •
edited by oliviertassinari
Loading

Problem description

When moving the anchor element (for example resizing the page), the popover doesn't reposition.

Link to minimally-working code that reproduces the issue

https://material-ui-1dab0.firebaseapp.com/component-demos/menus#menus

Versions

  • Material-UI: next
  • React: ^15.0.0
  • Browser: Any
@mbrookes mbrookes added component: Popover The React component. next labels Jan 15, 2017
@oliviertassinari oliviertassinari added the bug 🐛 Something doesn't work label Jul 4, 2017
@oliviertassinari oliviertassinari modified the milestone: v1.0.0-prerelease Jul 4, 2017
@stunaz stunaz mentioned this issue Jul 5, 2017
Closed
3 tasks
@slavab89 slavab89 mentioned this issue Jul 6, 2017
Closed
@oliviertassinari oliviertassinari added the good first issue Great for first contributions. Enable to learn the contribution process. label Jul 7, 2017
@oliviertassinari oliviertassinari removed this from the v1.0.0-prerelease milestone Jul 9, 2017
quiaro added a commit to quiaro/material-ui that referenced this issue Jul 19, 2017
@quiaro
[Popover] Doesn't reposition with anchorEl
77b4c52 
Closes mui#5937
quiaro added a commit to quiaro/material-ui that referenced this issue Jul 19, 2017
@quiaro
[Popover] Doesn't reposition with anchorEl
37db6d5 
Closes mui#5937
@quiaro quiaro mentioned this issue Jul 19, 2017
Merged
3 tasks
quiaro added a commit to quiaro/material-ui that referenced this issue Jul 19, 2017
@quiaro
[Popover] Doesn't reposition with anchorEl
f5c135a 
Closes mui#5937
quiaro added a commit to quiaro/material-ui that referenced this issue Jul 19, 2017
@quiaro
[Popover] Doesn't reposition with anchorEl
d544024 
Closes mui#5937
oliviertassinari pushed a commit that referenced this issue Jul 19, 2017
@quiaro @oliviertassinari
[Popover] Doesn't reposition with anchorEl (#7479)
ebcdeb8 
Closes #5937
@mbixby
Copy link

mbixby commented Aug 4, 2017

@oliviertassinari would this project accept integration http://tether.io ?

  • Optimized GPU-accelerated repositioning for 60fps scrolling
  • Reliable positioning on any possible corner, edge or point in between.
  • Support for repositioning or pinning the element when it would be offscreen
  • Designed to be embeddable in other libraries

@oliviertassinari
Copy link
Member

@mbixby I don't know, I haven't seen a situation on the v1-beta branch where we need tether yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐛 Something doesn't work component: Popover The React component. good first issue Great for first contributions. Enable to learn the contribution process.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mbrookes @mbixby @oliviertassinari