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.

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

css overflow on the popper = bad placement #143

Closed
ocombe opened this issue Jan 12, 2017 · 0 comments · Fixed by #146
Closed

css overflow on the popper = bad placement #143

ocombe opened this issue Jan 12, 2017 · 0 comments · Fixed by #146
Labels
bug Something is not working.

Comments

@ocombe
Copy link

ocombe commented Jan 12, 2017

CodePen demo

http://codepen.io/anon/pen/egZyZQ

Steps to reproduce the problem

  1. add the css property overflow: auto (or any other overflow value) on your popper div
  2. set the placement option to any value (ex: "left")
  3. the popper is always at the bottom left, whatever the value of placement. It gets even worse on "fixed" poppers because it will stay at the top of the window

What is the expected behavior?

Using placement should place the popper, even if it has an overflow set

What went wrong?

The popper isn't positionned because of the "overflow" css property :'(

Any other comments?

By the way, the link to the popper.js lib in your codepen example (given in the issue template) doesn't work: http://rawgit.com/FezVrasta/popper.js/master/src/popper.js --> 404
I used unpkg.com instead: https://unpkg.com/popper.js@1.0.2

@FezVrasta FezVrasta added the bug Something is not working. label Jan 12, 2017
FezVrasta pushed a commit that referenced this issue Jan 13, 2017
@FezVrasta FezVrasta mentioned this issue Jan 13, 2017
FezVrasta pushed a commit that referenced this issue Jan 13, 2017
FezVrasta pushed a commit that referenced this issue Jan 13, 2017
FezVrasta added a commit that referenced this issue Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants