Closed Bug 1673618 Opened 5 years ago Closed 5 years ago

Image not loading instantly

Categories

(Core :: DOM: Core & HTML, defect, P2)

Firefox 82
defect

Tracking

()

RESOLVED INVALID
Tracking Status
firefox-esr78 --- disabled
firefox81 --- wontfix
firefox82 --- wontfix
firefox83 --- wontfix
firefox84 --- wontfix

People

(Reporter: mahfouz.saif.elyazal, Unassigned)

References

(Regression)

Details

(Keywords: parity-chrome, regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:82.0) Gecko/20100101 Firefox/82.0

Steps to reproduce:

load the website: https://dozendev.wpengine.com/

Actual results:

One of the images in the div with classes: "et_pb_section et_pb_section_2 et_section_regular" does not load
The image (url: https://dozendev.wpengine.com/wp-content/uploads/2020/09/Group-8.png) does not load unless the window is resized several times

Expected results:

The image should appear once the page is loaded

Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached file reduced.html
Attachment #9184164 - Attachment mime type: text/plain → text/html
Has Regression Range: --- → yes
Has STR: --- → yes

Reproducible with Nightly on Ubuntu 18.04. Works with Chrome.

:hiro: do you have time to take a closer look at the issue?

Flags: needinfo?(hikezoe.birchill)
Keywords: parity-chrome
Severity: -- → S2
Priority: -- → P2

This is invalid (or a website issue really). The image is out of the viewport, and has no width / height, so it's zero-sized. So it doesn't intersect the viewport and thus we don't load it.

It might be the case that bug 1673785 fixes this drive-by, because it will slightly increase the margin horizontally and may make it just so that it's in the viewport.

But the website shouldn't rely on a particular lazyload margin. The fix on the website is changing width="auto" height="auto" (which are invalid) to use a sensible default width so that it intersects the viewport.

It just so happens that Chrome uses huge margins and the image loads there for a particular screen size.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(hikezoe.birchill)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: