jsor / lity

Lightweight, accessible and responsive lightbox.
https://sorgalla.com/lity/
MIT License
1.16k stars 196 forks source link

Plugins not included in lity.js file in the dist folder #267

Open johnsfuller opened 3 years ago

johnsfuller commented 3 years ago

Documentation says to

Include the Lity javascript and css files and its dependencies in your HTML document

It does not mention that the plugins have to be manually included.

A lot of YouTube issues stem from people not knowing to manually include their needed plugins (in their case the youtube.js plugin).

In the dist folder, I suggest either including the plugins in the dist/lity.js and dist/lity.min.js files or updating documentation so users know a build process is required.

Kiseki commented 3 years ago

Thanks @johnsfuller, I wasn't sure why it didn't work until you pointed this out.