Page MenuHomePhabricator

Lazy loaded images: Copy `usemap` attribute for image maps
Open, NormalPublic3 Story Points

Description

Note: There are several lazy loaded images bugs T207929, T199351, T209183. We'll probably want to fix these 3 bugs together.

When lazy loading imagemaps lazy loader doesn't set usemap attribute, hence breaking image maps.

How to reproduce:
Go to some article that use imagemap (the image shouldn't be small image, and not in the first section - lazy loading is done only for large images) and compare the desktop and the mobile versions.

Example:

URL

Acceptance criteria

  • The usemap attribute is copied across as data-usemap in the placeholder
  • The usemap attribute is copied across when JS loads the image

Event Timeline

eranroz created this task.Nov 9 2018, 8:48 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 9 2018, 8:48 PM
Jdlrobson renamed this task from lazy loading images breaks imagemaps to Lazy loaded images: Copy `usemap` attribute for image maps.
Jdlrobson triaged this task as Normal priority.
Jdlrobson updated the task description. (Show Details)
Jdlrobson moved this task from To Triage to Triaged but Future on the Readers-Web-Backlog board.
Jdlrobson set the point value for this task to 3.Nov 27 2018, 5:50 PM