- Issue created by @chike
- 🇮🇩Indonesia gausarts
Thank you.
I would normally add spaces around locked elements using just CSS so to keep the scroll functionality as needed.
But your solution makes sense, too.
For blazy:3.0.7+, we have an internal method to detect touch devices,
$.isTouch()
with a callback. Since pseudo:hover
is not available in touch devices, perhaps we can use it:
https://git.drupalcode.org/project/blazy/-/blob/3.0.8/js/src/dblazy.js?r...Or just use
window.matchMedia()
either as listener, or direct method.But yours is fine, too, if the library considered pseudo
:active
which is available at touch devices, but not:hover
. - Status changed to Needs review
7 months ago 1:39am 8 June 2024 - Status changed to Needs work
7 months ago 2:33am 8 June 2024 - 🇮🇩Indonesia gausarts
On second thoughts, while fine for Slick/Splide displays, this might break static grid functionality.
What do you think?
- 🇳🇬Nigeria chike Nigeria
Thank you!
You know the best way to do it, I am not a programmer as you know.
Will be good if the breakpoint is made configurable so someone can choose the breakpoint they want zoom turned off.
- Status changed to Fixed
5 months ago 7:02am 11 August 2024 - 🇮🇩Indonesia gausarts
On third thoughts, this should be addressed by CSS like #2 and responsive image to add enough room by padding or image styles to not fill up the entire screen at any devices.
The library has mobile supports, anyway, thus hover and scroll is not an issue.
Automatically closed - issue fixed for 2 weeks with no activity.