flickity icon indicating copy to clipboard operation
flickity copied to clipboard

Lazyload picture

Open patrickschaper opened this issue 5 years ago • 10 comments

This merge request adds a desired feature: Lazy loading for picture/source elements. This feature is important as srcset attribute cannot handle pixel density and viewport width at the same time.

patrickschaper avatar Oct 06 '20 14:10 patrickschaper

This just saved me – thank you @yesman82! @desandro this should definitely be merged in order to support <source> elements.

marcamos avatar Nov 09 '20 18:11 marcamos

Thank you so much for this contribution. I plan on working on Flickity again in 2021. This feature will be evaluated for merging in then. Thank you for your patience.

desandro avatar Jan 13 '21 14:01 desandro

This fixes the issue for me. Thank you!

luksak avatar Jan 19 '21 17:01 luksak

As a side-note: I wasn't able to get this to work with flickity-fade. But that probably has to do with my lack of npm foo :smile:

luksak avatar Jan 19 '21 17:01 luksak

I'd strongly welcome if this feature would be merged into Flickity. We really need to use the picture tag instead of the img tag because we have to control when which image gets loaded as they have different aspect-ratios.

tobipch avatar Apr 28 '21 08:04 tobipch

Any info on this? @desandro Thanks!

Samuell1 avatar Jun 30 '21 18:06 Samuell1

Used this update today to help speed up a client site using picture tag for displaying different images for mobile/desktop. This should definitely make it into the main branch!

5BYFIVE avatar Jul 06 '21 14:07 5BYFIVE

It's been 5 years since original issue --> https://github.com/metafizzy/flickity/issues/161#issuecomment-493670392

Can we please merge this and create a release?

bansavage avatar Jul 27 '21 18:07 bansavage

I'd like to see this pull request merged so I can use the element to implement support for serving WebP images in supported browsers when using Flickity.

Cipscis avatar Aug 18 '21 21:08 Cipscis

Hi,

When this PR will get merged?

gauravmanerkar avatar Oct 19 '21 11:10 gauravmanerkar

I've tested this solution. Works great! Would love to see this getting merged soon.

juukie avatar Nov 22 '21 15:11 juukie