Open Bug 1751574 Opened 3 years ago Updated 2 years ago

outline of tag "a" rendered differently from Blink when it contains tag "img"

Categories

(Core :: Layout, defect, P3)

Firefox 96
x86_64
Linux
defect

Tracking

()

Webcompat Priority P3
Tracking Status
firefox-esr91 --- unaffected
firefox96 --- wontfix
firefox97 --- wontfix
firefox98 --- wontfix
firefox99 --- wontfix
firefox100 --- wontfix

People

(Reporter: ibragimovrinat, Unassigned)

References

(Regression)

Details

(Keywords: regression, testcase)

Attachments

(6 files)

Attached file a-outline-broken.html (deleted) —

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:96.0) Gecko/20100101 Firefox/96.0

Steps to reproduce:

Open attached "a-outline-broken.html"

Actual results:

Outline rectangle is too small.

Expected results:

Outline rectangle should be tracing image borders. It should encompass whole area where "a" is active.

OS: Unspecified → Linux
Hardware: Unspecified → x86_64
Attached image Fx95-Fx96-compare.png (deleted) —

Rendering comparison between Firefox 95 and Firefox 96.

Has Regression Range: --- → yes
Has STR: --- → yes
Component: Untriaged → Layout
Flags: needinfo?(emilio)
Keywords: regression, testcase
Product: Firefox → Core
Regressed by: 1739894
Flags: needinfo?(emilio)
Attached file Tweaked test-case (deleted) —

(In reply to Rinat from comment #0)

Outline rectangle should be tracing image borders. It should encompass whole area where "a" is active.

FWIW that's not what other browsers do either if I tweak your test-case a bit, as per the above.

I can't make much sense of what Blink is doing here tbh.

Attachment #9260303 - Attachment mime type: text/plain → text/html

So I think for now a safe and easy patch would be to just special-case inline frames to always include overflow areas (effectively reverting to our previous behavior), wdyt Daniel?

Flags: needinfo?(dholbert)

Yeah, that sounds fine to me.

(We should double-check what other browsers do for block-in-inline splits and abspos children inside of inline frames, and make sure the change doesn't cause us to take a step away from compatibility for those cases.)

Flags: needinfo?(dholbert)

Actually, WebKit behaves like us, so I think I'd rather wait for https://github.com/w3c/csswg-drafts/issues/6981 to get resolved to potentially change this.

Severity: -- → S3
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Summary: outline of tag "a" rendered unexpectedly when it contains tag "img" → outline of tag "a" rendered differently from Blink when it contains tag "img"

Set release status flags based on info from the regressing bug 1739894

Emilio, should we do something in FF until the spec is updated, or should we continue to wait for the spec update?

Flags: needinfo?(emilio)

I don't think we should change behavior again until the spec is clarified.

Flags: needinfo?(emilio)
Webcompat Priority: --- → ?

We're seeing some site breakage, but they're only minor visual issues, so setting P3 for the moment.

Webcompat Priority: ? → P3
Attached image image.png (deleted) —

I ran into this today at Fidelity (Mozilla's 401k retirement-account provider in the US).

STR there are to log in & view your 401k account, go to the "Investments" tab, then "Choose my investments", and then "Exchange multiple investments".

This puts you on a page with a "Start Your Rebalance" button which looks quite broken when clicked (which gives it focus before the page load happens). See attached screenshot.

Duplicate of this bug: 1808516
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: