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

Fix css/css-typed-om/the-stylepropertymap/properties/fill-opacity.html to match the latest specification #7855

Merged

Conversation

cdumez
Copy link
Contributor

@cdumez cdumez commented Dec 19, 2022

422c18c

Fix css/css-typed-om/the-stylepropertymap/properties/fill-opacity.html to match the latest specification
https://bugs.webkit.org/show_bug.cgi?id=249588

Reviewed by Cameron McCormack.

Fix css/css-typed-om/the-stylepropertymap/properties/fill-opacity.html to match
the latest specification:
- https://svgwg.org/svg2-draft/painting.html#FillOpacity

The specification allows percentages, not just numbers.

* LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/fill-opacity-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/fill-opacity.html:

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

5755493

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

@cdumez cdumez self-assigned this Dec 19, 2022
@cdumez cdumez added the CSS Cascading Style Sheets implementation label Dec 19, 2022
@cdumez
Copy link
Contributor Author

cdumez commented Dec 19, 2022

Upstream PR: web-platform-tests/wpt#37580

@cdumez cdumez added the merge-queue Applied to send a pull request to merge-queue label Dec 19, 2022
…l to match the latest specification

https://bugs.webkit.org/show_bug.cgi?id=249588

Reviewed by Cameron McCormack.

Fix css/css-typed-om/the-stylepropertymap/properties/fill-opacity.html to match
the latest specification:
- https://svgwg.org/svg2-draft/painting.html#FillOpacity

The specification allows percentages, not just numbers.

* LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/fill-opacity-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/fill-opacity.html:

Canonical link: https://commits.webkit.org/258096@main
@webkit-commit-queue
Copy link
Collaborator

Committed 258096@main (422c18c): https://commits.webkit.org/258096@main

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

@webkit-early-warning-system webkit-early-warning-system merged commit 422c18c into WebKit:main Dec 19, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS Cascading Style Sheets implementation
Projects
None yet
4 participants