Closed SicorDev closed 6 years ago
Thanks for the feedback. Will take it under consideration, no conflict jQuery makes sense. Breakpoint configuration in template constants makes sense. For the next version I think I'll drop these things in.
Both features are available in the version 1.2.4 now!
Thank you very much!
Hi, nice extension - thank you very much!
We had to replace $( with jQuery( in youtubevideo.js to make it work, maybe you want to change that in a future release as well.
We also moved the "small" CSS breakpoint from 600 to 400, as it's still looking good for us.
Maybe you can make the breakpoint location configurable. Or add a plugin option / constant to manually decide wether it should add "small" or not.
Greetings, Manuel