search

dimsemenov / Magnific-Popup

Light and responsive lightbox script with focus on performance.
http://dimsemenov.com/plugins/magnific-popup/
MIT License
11.38k stars 3.49k forks source link

Close button's cursor #252

Open zsitro opened 10 years ago

zsitro commented 10 years ago

Hi, is it intentional to modify close button's cursor face from pointer to something else like a magnifier?

imho, it should always be a pointer. But I would accept a reasonable explanation :)

dimsemenov commented 10 years ago

Hi,

Yep, it was intentional. As button is stretched to 100% of image width (to create clickable gap on top that closes the popup), making whole top bar with cursor:pointer is even more weird.

But you're probably right, in next update I'll add another container inside button and make just the icon with cursor:pointer. I'm not sure if I remember why I didn't do this initially...

AndreiMaz commented 9 years ago

Hi,

Any update on it?

townivan commented 8 years ago

Meanwhile, this css may help:

.mfp-zoom-out-cur { cursor: pointer; } .mfp-zoom-out-cur .mfp-image-holder .mfp-close { cursor: pointer; }

ivangretsky commented 8 years ago

There is still a need for a fix to this. The best popup there is just should have this thing figured.