sydcanem / bootstrap-contextmenu

Context menu plugin for Twitter's Bootstrap framework
http://sydcanem.com/bootstrap-contextmenu/
645 stars 193 forks source link

Might want to bump up the z-index on the menu #16

Closed JohnnyBurbank closed 10 years ago

JohnnyBurbank commented 11 years ago

Again, not sure if this will affect anyone negatively, but I have a lot of absolutely positioned divs with z-indexes, and I'll need the context menu to always be above all others. Might be reasonable to add a z-index to the menu of say 9999, as I imagine most people would have z-indexes much lower than that, but would always want the menu on top. Thx again!

dariusclay commented 11 years ago

I would agree with that. Most other tools set a large z-index on elements that should appear above all others, like fancybox or select2 On Apr 2, 2013 4:10 PM, "JohnnyBurbank" notifications@github.com wrote:

Again, not sure if this will affect anyone negatively, but I have a lot of absolutely positioned divs with z-indexes, and I'll need the context menu to always be above all others. Might be reasonable to add a z-index to the menu of say 9999, as I imagine most people would have z-indexes much lower than that, but would always want the menu on top. Thx again!

— Reply to this email directly or view it on GitHubhttps://github.com/sydcanem/bootstrap-contextmenu/issues/16 .