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

[LBSE] Not all 'outermost' <svg> elements enforce a stacking context #4157

Conversation

nikolaszimmermann
Copy link
Contributor

@nikolaszimmermann nikolaszimmermann commented Sep 9, 2022

a57723e

[LBSE] Not all 'outermost' <svg> elements enforce a stacking context
https://bugs.webkit.org/show_bug.cgi?id=244965

Reviewed by Rob Buis.

Add missing condition in Style::Adjuster::adjustSVGElementStyle() to enforce
a stacking context if the element that receives the style is an outermost
<svg> element (or acts as such). It is mentioned in the SVG2 spec note just
a few lines above, but not implemented yet.

* LayoutTests/platform/mac-monterey-wk2-lbse-text/svg/css/text-shadow-multiple-expected.txt:
* LayoutTests/platform/mac-monterey-wk2-lbse-text/svg/custom/image-parent-translation-expected.txt:
* LayoutTests/platform/mac-monterey-wk2-lbse-text/svg/custom/second-inline-text-expected.txt:
* Source/WebCore/style/StyleAdjuster.cpp:
(WebCore::Style::Adjuster::adjustSVGElementStyle):

Canonical link: https://commits.webkit.org/254314@main

abb2f35

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-debug βœ… πŸ›  gtk βœ… πŸ›  wincairo
βœ… πŸ§ͺ webkitperl βœ… πŸ§ͺ ios-wk2 βœ… πŸ›  mac-AS-debug βœ… πŸ§ͺ gtk-wk2
βœ… πŸ§ͺ api-ios βœ… πŸ§ͺ api-mac βœ… πŸ§ͺ api-gtk
βœ… πŸ›  tv   πŸ§ͺ mac-wk1
βœ… πŸ›  tv-sim βœ… πŸ§ͺ mac-wk2
βœ… πŸ›  πŸ§ͺ merge loading πŸ›  watch   πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  watch-sim βœ… πŸ§ͺ mac-wk2-stress

@nikolaszimmermann nikolaszimmermann self-assigned this Sep 9, 2022
Copy link
Contributor

@rwlbuis rwlbuis left a comment

Choose a reason for hiding this comment

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

LGTM.

@nikolaszimmermann nikolaszimmermann added the merge-queue Applied to send a pull request to merge-queue label Sep 9, 2022
https://bugs.webkit.org/show_bug.cgi?id=244965

Reviewed by Rob Buis.

Add missing condition in Style::Adjuster::adjustSVGElementStyle() to enforce
a stacking context if the element that receives the style is an outermost
<svg> element (or acts as such). It is mentioned in the SVG2 spec note just
a few lines above, but not implemented yet.

* LayoutTests/platform/mac-monterey-wk2-lbse-text/svg/css/text-shadow-multiple-expected.txt:
* LayoutTests/platform/mac-monterey-wk2-lbse-text/svg/custom/image-parent-translation-expected.txt:
* LayoutTests/platform/mac-monterey-wk2-lbse-text/svg/custom/second-inline-text-expected.txt:
* Source/WebCore/style/StyleAdjuster.cpp:
(WebCore::Style::Adjuster::adjustSVGElementStyle):

Canonical link: https://commits.webkit.org/254314@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/LBSE-Not-all-outermost-svg-elements-enforce-a-stacking-context branch from abb2f35 to a57723e Compare September 9, 2022 18:29
@webkit-commit-queue
Copy link
Collaborator

Committed 254314@main (a57723e): https://commits.webkit.org/254314@main

Reviewed commits have been landed. Closing PR #4157 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit a57723e into WebKit:main Sep 9, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Sep 9, 2022
@nikolaszimmermann nikolaszimmermann deleted the eng/LBSE-Not-all-outermost-svg-elements-enforce-a-stacking-context branch November 5, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SVG For bugs in the SVG implementation.
Projects
None yet
4 participants