search

Angrycreative / bj-lazy-load

WordPress plugin for lazy loading
GNU General Public License v2.0
40 stars 25 forks source link

Disable for Print not working #23

Open kylelarkin opened 8 years ago

kylelarkin commented 8 years ago

I noticed in Version 0.6.4 you added the feature of disabling when printing. For some reason, when I print preview in both Chrome and Firefox, only the images that have been loaded already in the browser show up in the preview.

By adding a class of 'no-lazy' to the first image on the page, it somehow fixes the issue and all images on the page show in the print preview.

Here's an example of a page that does not have no-lazy applied: https://placesjournal.org/article/camp-code/ And here's one that does: https://placesjournal.org/article/multilevel-metropolis-urban-skyways/

I'm sure it could be something conflicting on my end but I thought I would ask here if you had seen the issue before.

Thanks!

jan-san commented 5 years ago

Confirmed that this issue still exists. It could be fixed by registering a listener for the onbeforeprint event and load all remaining lazy resources on the page (also add the polyfill for Safari).