The installation instructions require you to copy just the lozad.min.js file and place it into /libraries/lozad, however the actual lozad distribution package places the minified js file under /dist.
Normally this wouldn't be a problem because you can manually move that file to the expected location, but if you're managing your Drupal site with Composer and installing js libraries via Asset Packagist, you don't really get to control where the files end up. Because of this, I think it would be better for the module to look for the minified js at the path defined by the lozad package.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | image_lazy_loader_lozad_under_dist-3050772-1.patch | 1.46 KB | amklose |
Comments
Comment #2
amkloseHere's a patch I made that allows me to just require the lozad library with Composer. I edited the libraries file which actually points to the minified js, the install file which is checking to see if the js exists before installing the module, and the README file to slightly modify the installation instructions.
Comment #3
psebborn commentedComment #5
psebborn commentedThanks for this, good spot! I've added this now, will be in the next release
Comment #6
psebborn commentedComment #7
psebborn commentedComment #8
psebborn commented