2019610 - Inner SVG with overflow auto is hiding overflowing content (behavior change from bug 1999100)

Status: RESOLVED FIXED

(Core :: SVG, defect, P3)

Description

[Tiberiu Zuld]

Attachment: svg-overflow.png

Steps to reproduce:

SVG overflow: auto is no longer working as expected in v148 compared with v147.
In Chrome and Safari it is working as expected.

I think this is triggered by changes for https://bugzil.la/1999100

Example:
https://jsfiddle.net/7nzgbfa2/2/

In Chrome the overflow of the red rectangle is shown, but in FF v148 is not shown.
It was working correctly in v147.

Actual results:

Rect element overflowing parent SVG element is clipped when SVG has overflow: auto

Expected results:

Rect element overflowing parent SVG element shouldn't be clipped when SVG has overflow: auto

Comment 1

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

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

Comment 2

[Emilio Cobos Álvarez [:emilio]]

That seems... expected? Generally overflow values that apply clipping do clip overflow, it seems like a bug that it didn't before tbh.

I guess WebKit also behaves like chrome here so we could consider changing back, but not sure the spec is clear about that.

Comment 3

[Emilio Cobos Álvarez [:emilio]]

mozregression agrees this was caused by bug 1999100.

Comment 4

[Emilio Cobos Álvarez [:emilio]]

https://hg-edge.mozilla.org/mozilla-central/rev/e81fe51adc7c48e454bc04b7d82a6cf2211ebe82#l7.159 is the line that causes this behavior change. But I think this is the right thing to do.

Comment 5

[Emilio Cobos Álvarez [:emilio]]

Attachment: Test-case that compares overflow: auto on inner vs. outer SVG

I don't see why these two should render differently?

Comment 6

[Emilio Cobos Álvarez [:emilio]]

This seems kinda broken: https://svgwg.org/svg2-draft/render.html#OverflowAndClipProperties

Comment 7

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 2019610 - Add a test for overflow: auto on inner vs. outer SVG. r=#svg!

See also https://github.com/w3c/svgwg/issues/1072

Comment 8

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

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

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/7b1a35d6c70c
https://hg.mozilla.org/integration/autoland/rev/42c3296d9fdb
Add a test for overflow: auto on inner vs. outer SVG. r=firefox-svg-reviewers,dholbert

Comment 10

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

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

Comment 11

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/42c3296d9fdb

Comment 12

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

Upstream PR merged by moz-wptsync-bot