Page MenuHomePhabricator

Lazy loaded images: Images may be cut off by 2px
Closed, DuplicatePublic

Description

See also T135430.

The added border should increase the layout box of the image by 2 pixels. However in an effort to avoid layout differences the image layout was given box-sizing: border-box; (which prevents the increase in layout size) and overflow: hidden; (which hides the excess image data of the right and bottom edge).

This however effectively deforms the image which is wrong.

This bug is currently invisible due to the image not being inserted until after it completes loading, at which point the border is hidden by the loaded class. However after T135430 is fixed, this will likely become an issue.

Related Objects

Event Timeline

Krinkle created this task.May 16 2016, 7:46 PM
Restricted Application added a subscriber: Zppix. · View Herald TranscriptMay 16 2016, 7:46 PM

Change 289013 had a related patch set uploaded (by Krinkle):
Lazy-load: Remove the placeholder once the image has loaded

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

Jdlrobson raised the priority of this task from Normal to Needs Triage.May 17 2016, 4:55 PM

Team should prioritise in standup.

phuedx triaged this task as Normal priority.May 17 2016, 5:14 PM