elevatezoom-plus icon indicating copy to clipboard operation
elevatezoom-plus copied to clipboard

Need option to specify z-index

Open teej043 opened this issue 8 years ago • 1 comments

It's a nice to have since this plugin doesn't need much custom css, so rather than putting

.zoomWindowContainer > div{
  z-index: 100!important;
}

to the stylesheet, why not have this option on the initialization itself?

teej043 avatar Aug 29 '16 12:08 teej043

@teej043 there is currently a zIndex option that applies to the inner zoom zoomWindow and the specified container (by default ZoomContainer). The same option applies to the z-index for both classes, which may or may not be working as designed.

I'm not sure which behavior makes sense. If zIndex is set at the container-level, should child divs inherit from that? Or should the container and child divs have separately controlled zIndex values?

In my mind, it makes sense that all child divs would inherit from the parent container. This would be a relatively easy change to make I think.

kalebanderson avatar Apr 09 '18 20:04 kalebanderson