WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
248152
[web-animations] keyframes should be recomputed when a parent element changes value for a non-inherited property set to "inherit"
https://bugs.webkit.org/show_bug.cgi?id=248152
Summary
[web-animations] keyframes should be recomputed when a parent element changes...
Antoine Quint
Reported
2022-11-20 11:37:38 PST
We are failing a number of WPT tests where a keyframes uses an "inherit" value for a given property and that property changes on a parent dynamically. For instance, in web-animations/responsive/perspective.html: test(function() { container.style.perspective = 'none'; var player = element.animate([{perspective: 'inherit'}, {perspective: '20px'}], 4000); player.pause(); player.currentTime = 1000; assert_equals(getComputedStyle(element).perspective, 'none'); container.style.perspective = '100px'; assert_equals(getComputedStyle(element).perspective, '80px'); container.style.perspective = 'none'; assert_equals(getComputedStyle(element).perspective, 'none'); }, 'perspective responsive to inherited changes'); This makes us fail a number of similar WPT tests under web-animations/responsive/.
Attachments
Test
(694 bytes, text/html)
2022-11-20 11:45 PST
,
Antoine Quint
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Antoine Quint
Comment 1
2022-11-20 11:45:34 PST
Created
attachment 463630
[details]
Test Adding a standalone test using a CSS animation.
Antoine Quint
Comment 2
2022-11-20 11:47:05 PST
Currently this seems to only work in Chrome (Canary anyway).
Radar WebKit Bug Importer
Comment 3
2022-11-27 11:38:16 PST
<
rdar://problem/102694235
>
Antoine Quint
Comment 4
2023-01-31 06:11:35 PST
Pull request:
https://github.com/WebKit/WebKit/pull/9381
EWS
Comment 5
2023-01-31 14:06:08 PST
Committed
259645@main
(25ead8e66edd): <
https://commits.webkit.org/259645@main
> Reviewed commits have been landed. Closing PR #9381 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug