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.Nov 12 2018, 7:15 PM
Jdlrobson triaged this task as Normal priority.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Readers-Web-Backlog.
Jdlrobson moved this task from To Triage to Triaged but Future on the Readers-Web-Backlog board.
Jdlrobson updated the task description. (Show Details)Nov 27 2018, 5:43 PM
Jdlrobson set the point value for this task to 3.Nov 27 2018, 5:50 PM