1790167 - AFL Logo's not shown

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Greg ( Hank1 )]

Attachment: Screenshot 2022-09-10 101900.png

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/107.0.0.0 Safari/537.36

Steps to reproduce:

go to https://afl.com.au/

Actual results:

the football club logo's dont show on the top of the website

Expected results:

as shown in the picture those logo's should show

Comment 1

[Greg ( Hank1 )]

Attachment: Screenshot 2022-09-10 102700.png

hope that helps with another fullscreen screenshot of what it looks like an what it should look like, those logo's should be showing up the top of the page , as shown in picture 1

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Graphics' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 3

[Greg ( Hank1 )]

Attachment: Screenshot 2022-09-10 103213.png

this screenshot taken in Google Chome is what it should look like in Firefox Nightly

Comment 4

[Alice0775 White]

Reproduced if the browser window width > approx. 1380px.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=b6d4cc392199eac68b7641fe6ce2be73a84381f6&tochange=76c1fb5130dcd9f4562826d42194ebf74ca268c8

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

:TYLin, since you are the author of the regressor, bug 1340715, could you take a look? Also, could you set the severity field?

For more information, please visit auto_nag documentation.

Comment 6

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1790167 - Make percent isize also fall back to fallback intrinsic size. r=TYlin

Matches other browsers and makes sense that width="100%" would behave
the same as width: 100% via CSS, I guess.

Comment 7

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1790167 - Simplify aspect-ratio code for outer svg. r=TYLin

Drive-by, but this should be more correct, since I'm not sure how this
code would handle mSVGView otherwise, which is in effect on the relevant
page.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3f3f7b8d088e
Make percent isize also fall back to fallback intrinsic size. r=TYLin

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/35867 for changes under testing/web-platform/tests

Comment 10

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Thank you Emilio for fixing this regression!

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e1af36b99512
Simplify aspect-ratio code for outer svg. r=TYLin

Comment 12

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/3f3f7b8d088e
https://hg.mozilla.org/mozilla-central/rev/e1af36b99512

Comment 13

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot