clu3 / bootstro.js

Tiny JS library using bootstrap's popovers to help guide your users around your website
http://clu3.github.io/bootstro.js/
MIT License
1.33k stars 235 forks source link

Fix issue with backdrop not displaying properly with iscroll and other libraries which use transform to change the z-index. #29

Open edmenendez opened 11 years ago

clu3 commented 11 years ago

Hi @edmenendez Thanks alot for the pull request, I have 2 points

  1. I don't know about iScroll and I'm not sure what you mean by z-index confusion, but could it be simpler by specifying a zIndex for the bootstrap overlay so they won't conflict?
  2. I completely agree that there should be a "showBackdrop" setting

Cheers

edmenendez commented 11 years ago

It's related to WebKit bug 61824 and various people have run into it as seen in StackOverflow :-1:

iScroll happens to be one package that uses transform z-index, but definitely not the only one. My solution is simply a workaround to avoid the problem for now by creating the layer out of four boxes instead of a single clean layer like you have it only if you pass useBackdropMethod2 true.

clu3 commented 11 years ago

Sorry @edmenendez i'd love to merge your pull request but I feel like it will make this little utility more complex. I want to keep it really simple. However, I'll mention your request and fork in the README and demo docs as soon as I can. Thanks a lot for your input.