Page MenuHomePhabricator

Do not lazy load images on main page
Closed, ResolvedPublic1 Story Points

Description

It is unnecessary and currently broken in that it leads to a repaint. Let's avoid running on this page.

Event Timeline

Jdlrobson created this task.May 4 2016, 7:51 PM
Restricted Application added subscribers: Zppix, TerraCodes, Aklapper. · View Herald TranscriptMay 4 2016, 7:51 PM
Jdlrobson set the point value for this task to 1.May 9 2016, 4:15 PM
Jdlrobson triaged this task as Normal priority.May 9 2016, 4:58 PM

Change 287670 had a related patch set uploaded (by Jdlrobson):
Add lazy loading images test for main page

https://gerrit.wikimedia.org/r/287670

Looking at this closely it seems that this bug is invalid. I've written a test that shows that section wrapping and lazy loaded images will not run on the main page.

I think possibly what happened here is an inline image without a height was in the main page causing a repaint. It wasn't related to lazy loaded images.

Thus removing this as a blocker.

Change 287670 merged by jenkins-bot:
Add lazy loading images test for main page

https://gerrit.wikimedia.org/r/287670