Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#28903] - Lazy loading properly

NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
2 May 2020

Pull Request alternative to #28838 .

Summary of Changes

Properly introduce the loading=lazy attribute for the images

Testing Instructions

Expected result

Actual result

Documentation Changes Required

avatar dgrammatiko dgrammatiko - open - 2 May 2020
avatar dgrammatiko dgrammatiko - change - 2 May 2020
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 May 2020
Category ⇒ Layout
avatar dgrammatiko dgrammatiko - change - 2 May 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 May 2020
Category Layout ⇒ JavaScript Repository NPM Change Layout
avatar wilsonge wilsonge - close - 2 May 2020
avatar wilsonge wilsonge - merge - 2 May 2020
avatar wilsonge wilsonge - change - 2 May 2020
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2020-05-02 15:50:26
Closed_By ⇒ wilsonge
Labels Added: NPM Resource Changed
avatar wilsonge
wilsonge - comment - 2 May 2020

This obviously fine independently of #28838

Thanks!

avatar C-Lodder
C-Lodder - comment - 2 May 2020

@wilsonge I can't see using a regex to parse a potentially large DOM string, a good solution. This is great, it solves several aspects.
As for the content, adding the attribute on page load is pointless. Do it when the content is saved

Add a Comment

Login with GitHub to post a comment