-
Notifications
You must be signed in to change notification settings - Fork 942
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
Popover shown below the navbar-fixed-top #125
Comments
👍 |
Actually I was experiencing a similar problem in my application. This happens when placement is |
If you try this on a mobile device, the issue is compounded by the anchor element for the popover often disappearing off screen if the placement of the popover is "below" and half the popover being hidden by the navbar if the placement is "above". If I might make a suggestion on this issue -- considering that the anchor element for the popover can be of varying size, the display height will vary, and the position of the popover relative to the anchor element can vary -- rather than trying to solve precise positioning for each possible scenario, why not handle the positioning issue the way ZURB's Joyride seems to, and just scroll such that the popover is in the middle of the viewport, rather than at the top? The z-index fix will help for when the anchor is at the top of the page, but for anchors lower in a page, that fix will constantly render the navbar navigation inaccessible on mobile devices (due to the popover blocking the controls) without a "midline-scroll" as described above. Edit: You can see a sample scroll function to do this in my comment on #114 |
close, done in v0.6.0 following #132 implementation. |
this is the result. i'm quite sure it's like this after the upgrade to support twitter bootstrap 3:
to solve it, we can set a higher
z-index
for the popovers with classtour
.@emmenko and @sorich87, looking forward to hear from you if you agree on the fix.
The text was updated successfully, but these errors were encountered: