It would be nice if you included Beta versions in the required version range - makes the testing easier. I had to modify it myself.
In package-info.xml the name and destination of the JavaScript file is incorrect. So it doesn't install unless you fix those.
Also I'm not sure if lazy loading all the images on the page is a good idea. Maybe you should just limit it to images in posts, signatures and avatars.
Oook. I should not change my mind so often... :-\
Should be fixed now. Thanks! (And sorry for the troubles. :-[)
Yup, the first idea was avatars and attachments, then I wend for "everything" just because it was easier. lol
At the moment is more a childish game, rather than something really useful. :P
I'll play some more in the next days. ;D
You don't seem to have THEMEDIR, so use $themedir. You can also just set the range as 1.0 beta 2 - 1.0.99; compareVersions() can handle that. (guess who coded it! :P)
For avatars, posts and signatures, you can just look for classes "avatar" and "bbc_image". Attachments don't seem to have a class attached to them but you could probably use the ";image" bit to catch them. (it would have been nice if they had a class though)
I thought we changed for all-caps the theme dir as well.... meh.
/me opens a ranting topic.
I don't know why, but this brought my forum to a crawl. Uninstalled it and everything is snappy again.
"Nice" to know.
It may be related to the buffer-rewriting it does in order to replace the images... I have to check it out more.
Does this still work with Elkarte 1.1.1 ?
It looks to me like it very well might. Why don't you give it a try?
It works fine on 1.1.1. ;)
I've made a PR to the official repo, but until that makes it through the system, here is a minor update
- Fix admin icons being hidden
- prevent it from deferring images that were already browser deferred (a 1.1.9 update)
- update from the 1.9.3 to 1.9.7 lib and defer it loading
- allow standard 1.1 install
- http->https in a few areas