Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

popover positioning is wrong for large(r) values of .popover max-width #1236

Closed
facboy opened this issue Nov 7, 2013 · 9 comments
Closed

Comments

@facboy
Copy link

facboy commented Nov 7, 2013

it seems that if .popover max-width is set too large (eg over 350px) that the positioning relative to the trigger element is wrong. i couldn't quite work out what was causing it. setting a fixed width (rather than max-width) seems to fix the issue.

@pkozlowski-opensource
Copy link
Member

@facboy could please provide a reproduce scenario using http://plnkr.co/? I'm not sure I understand the problem just by reading your description.

@facboy
Copy link
Author

facboy commented Nov 10, 2013

maybe it's because i'm using bootstrap 3: http://plnkr.co/edit/FeG78i?p=info. i can get it to happen in the preview page in 2.3.2, or in fullscreen but only by setting the max-width infeasibly large: http://plnkr.co/edit/ic5ejB?p=info

@pkozlowski-opensource
Copy link
Member

Looks very similar to #944, to be checked when it gets fixed

@pkozlowski-opensource
Copy link
Member

@facboy sorry for being pain in a neck, but for the life of me I'm not sure what is the issue with BS3 - for me things work OK in chrome. What is a browser you are using?

Could you please attach a picture of a broken popover? Putting here the latest version of the code from the bootstrap3 branch.

@ToucheSir
Copy link

Just tested @facboy's plunk. I have the same problem (using Firefox 26):
http://plnkr.co/edit/nT94QHl75qpwAHDQYyLJ?p=preview

Edit: Tested with a pull from the bootstrap3 branch, and still no dice. Changing the spans to block elements appears to make things work again, but only when the element spans the whole width of the column....

@facboy
Copy link
Author

facboy commented Nov 24, 2013

i'm using chrome 31:
plunk

@janhoogeveen
Copy link

Got the same issue here as well. Issue occurred when my popover was positioned to the right side of my screen and would cause an overflow. The max-width style would shrink the popover and in doing so the popover would be positioned much higher and to the right than it should.

Simply gave up trying to fix this and using a fixed width now. Much more reliable :).

@jeez727
Copy link

jeez727 commented Dec 17, 2014

It's working fine. Thank you janhoogeveen

@rvanbaalen
Copy link
Contributor

This is fixed when using the most recent ui-bootstrap and bootstrap css.

http://plnkr.co/edit/0s6RbB?p=preview

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants