[webkit-changes] [WebKit/WebKit] f438e8: Animation using padding-block doesnt override set ...
Antoine Quint
noreply at github.com
Tue Jul 11 11:39:02 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: f438e854500a424cbcef992bcfec923d0b7d26b8
https://github.com/WebKit/WebKit/commit/f438e854500a424cbcef992bcfec923d0b7d26b8
Author: Antoine Quint <graouts at webkit.org>
Date: 2023-07-11 (Tue, 11 Jul 2023)
Changed paths:
A LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-block-interpolation-expected.txt
A LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-block-interpolation.html
A LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-inline-interpolation-expected.txt
A LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-inline-interpolation.html
M Source/WebCore/animation/CSSPropertyAnimation.cpp
M Source/WebCore/style/Styleable.cpp
Log Message:
-----------
Animation using padding-block doesnt override set padding style
https://bugs.webkit.org/show_bug.cgi?id=258813
rdar://112023856
Reviewed by Tim Nguyen.
We did not support animating the padding-block shorthand. To do so we must do two things:
1. in CSSPropertyAnimation.cpp we must actually mark that shorthand as animatable.
2. in transitionMatchesProperty() we must support the possibility that a property
is a shorthand for logical properties.
I suppose we could make transitionMatchesProperty() re-entrant in some way such that shorthands
of shorthands could also work, but this will do for now.
We also add animation support for padding-inline since it's a closely-related shorthand.
* LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-block-interpolation-expected.txt: Added.
* LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-block-interpolation.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-inline-interpolation-expected.txt: Added.
* LayoutTests/imported/w3c/web-platform-tests/css/css-logical/animations/padding-inline-interpolation.html: Added.
* Source/WebCore/animation/CSSPropertyAnimation.cpp:
(WebCore::CSSPropertyAnimationWrapperMap::CSSPropertyAnimationWrapperMap):
* Source/WebCore/style/Styleable.cpp:
(WebCore::transitionMatchesProperty):
Canonical link: https://commits.webkit.org/265956@main
More information about the webkit-changes
mailing list