Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gecko Bug 1565557] ResizeObserver for SVG elements with CSS layout boxes use the layout boxes. #35437

Merged
merged 1 commit into from Aug 12, 2022

Conversation

moz-wptsync-bot
Copy link
Collaborator

As per the CSSWG resolution here: w3c/csswg-drafts#4032
Previously, any and all SVG elements used its SVG object bounding box. Now, SVG
elements that have their own CSS layout box use them. These match the following
CSS rules: svg:root, *:not(svg|*) > svg, svg|foreignObject > svg

Differential Revision: https://phabricator.services.mozilla.com/D154246

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1565557
gecko-commit: 742e5698d74fd51b30049622b9beaf963e8cefc0
gecko-reviewers: emilio

…boxes.

As per the CSSWG resolution here: w3c/csswg-drafts#4032
Previously, any and all SVG elements used its SVG object bounding box. Now, SVG
elements that have their own CSS layout box use them. These match the following
CSS rules: `svg:root, *:not(svg|*) > svg, svg|foreignObject > svg`

Differential Revision: https://phabricator.services.mozilla.com/D154246

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1565557
gecko-commit: 742e5698d74fd51b30049622b9beaf963e8cefc0
gecko-reviewers: emilio
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Firefox project.

@moz-wptsync-bot moz-wptsync-bot merged commit cd26ed8 into master Aug 12, 2022
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1565557 branch August 12, 2022 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants