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 Displays Behind Modal (3.1.1) #13216

Closed
thedavidscherer opened this issue Mar 28, 2014 · 2 comments · Fixed by #13217
Closed

Popover Displays Behind Modal (3.1.1) #13216

thedavidscherer opened this issue Mar 28, 2014 · 2 comments · Fixed by #13217
Labels
confirmed css
Milestone
v3.2.0

Comments

@thedavidscherer
Copy link

The issue is prevalent across all browsers. It is caused by the @zindex-popover and @zindex-tootip settings in variables.less.

jsBin reproducing the error: http://jsbin.com/tesaceso/3/edit?output

This issue has seemingly popped up repeatedly over the years going back to BS 2 (that I can tell).

Changing the @zindex-tootip and @zindex-popover values to be higher than @zindex-modal resolves the issue.
@thedavidscherer thedavidscherer changed the title Popover Displays Behind Modal Popover Displays Behind Modal (3.1.1) Mar 28, 2014
@cvrebert
Copy link
Collaborator

Confirmed, after adding missing div.modal-body: http://jsbin.com/tesaceso/5

thedavidscherer pushed a commit to thedavidscherer/bootstrap that referenced this issue Mar 28, 2014
@dscherertfm
fixes twbs#13216
7dec5c7 
change `@zindex-popover` and `@zindex-tooltip` values to be greater than
those set for `@zindex-modal` allowing tooltips and popovers to be
displayed in front of modals (and all other content).
@thedavidscherer
Copy link
Author

@cvrebert My bad, sorry about that.

@thedavidscherer thedavidscherer mentioned this issue Mar 28, 2014
Merged
@juthilo juthilo added this to the v3.2.0 milestone Mar 29, 2014
@mdo mdo closed this as completed in #13217 Apr 3, 2014
mdo added a commit that referenced this issue Apr 3, 2014
@mdo
Merge pull request #13217 from thedavidscherer/fix-13216
a444863 
fixes #13216
@mdo mdo mentioned this issue Apr 3, 2014
Closed
stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
@dscherertfm
(less) fixes twbs#13216
235106e 
change `@zindex-popover` and `@zindex-tooltip` values to be greater than
those set for `@zindex-modal` allowing tooltips and popovers to be
displayed in front of modals (and all other content).
stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
@mdo
(less) Merge pull request twbs#13217 from thedavidscherer/fix-13216
2ce66b5 
fixes twbs#13216
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
confirmed css
Projects
None yet
Milestone
v3.2.0
Development

Successfully merging a pull request may close this issue.

fixes #13216 thedavidscherer/bootstrap
3 participants
@cvrebert @thedavidscherer @juthilo