Lazysizes: Get rid of lazyload class

Created on 8 Mar 2017 · 2Comments · Source: aFarkas/lazysizes

Hello,

Love this plugin, however I'm not fan of adding a class on each imagess to apply laryload on them.

It could be done by targeting image with data-src or data-srcset attributes directly.

Source

Zmove

👍2

Most helpful comment

My two cents on this:

class-based selectors outperforms data attribute matching,
lazyload class name can be reconfigured via JS API options to e.g. "lazy" or "lz" and so on if the size of html code matters, or if it's easier to be used everytime,
I like to style my lazy loaded images, mostly to add opacity transition effects.

So I find it beneficial the way it is now.

jurajkapsz on 8 Mar 2017

👍4

All 2 comments

My two cents on this:

class-based selectors outperforms data attribute matching,
lazyload class name can be reconfigured via JS API options to e.g. "lazy" or "lz" and so on if the size of html code matters, or if it's easier to be used everytime,
I like to style my lazy loaded images, mostly to add opacity transition effects.

So I find it beneficial the way it is now.

jurajkapsz on 8 Mar 2017

👍4

It could be done by targeting image with data-src or data-srcset attributes directly.

Yeah, but what about scripts, iframes? That kind of stuff adds code complexity and starts to make assumptions about data structure that could have unexpected consequences. For example, WordPress may add data-src AND src automatically. But it doesn't add the lazyload class.