255114 2023-04-06 13:43:41 -0700 Wave animation on https://deploy-preview-14--unas-new-site.netlify.app/ doesn't work in Safari 2025-10-16 23:43:39 -0700 1 1 1 Unclassified WebKit Animations WebKit Nightly Build Unspecified Unspecified RESOLVED CONFIGURATION CHANGED https://bugs.webkit.org/show_bug.cgi?id=173154 https://bugs.webkit.org/show_bug.cgi?id=111927 https://bugs.webkit.org/show_bug.cgi?id=296048 https://deploy-preview-14--unas-new-site.netlify.app/ BrowserCompat, InRadar P2 Normal --- 1 simon.fraser webkit-unassigned ahmad.saleem792 dino graouts graouts karlcow koivisto sabouhallawa simon.fraser una.kravets webkit-bug-importer oldest_to_newest 1947106 0 simon.fraser 2023-04-06 13:43:41 -0700 Load https://deploy-preview-14--unas-new-site.netlify.app/ in Firefox and note the nice wave animation on the lower edge of the header. Now load in Safari; the wave animation is not there. 1947107 1 webkit-bug-importer 2023-04-06 13:44:04 -0700 <rdar://problem/107725408> 1947108 2 simon.fraser 2023-04-06 13:44:57 -0700 The animation is driven by: .wave:where(.astro-J7PV25F6) { display:block; width:100%; height:clamp(2rem, 5vw, 10rem); margin:0; z-index:0; bottom:0 } #page-divide:where(.astro-J7PV25F6) { animation:moveforward 10s linear infinite } @keyframes moveforward { 0% { transform:translate(-90px) } to { transform:translate(85px) } } 1947109 3 simon.fraser 2023-04-06 13:46:20 -0700 Looks like we fail to animation the <path> inside the <defs> 1947238 4 graouts 2023-04-07 00:21:11 -0700 Presumably a <path> within a <defs> does not have a renderer, and I guess we bail out of the accelerated animation code which does not account for this possibility. 1947320 5 graouts 2023-04-07 10:25:21 -0700 Actually, we correctly animate the transform. This is probably something SVG-specific about updating the shadow tree rendering. 1947321 6 465812 graouts 2023-04-07 10:32:24 -0700 Created attachment 465812 Test 1947322 7 graouts 2023-04-07 10:35:19 -0700 I think we need to call a method on SVGElement to notify that the style changed due to an animation which would then call invalidateInstances(). 1947326 8 graouts 2023-04-07 10:40:41 -0700 I wonder if we should add a new virtual method to StyledElement to notify when the animated style changed which would allow SVGElement to override. We could then call this method at the end of Style::TreeResolver::createAnimatedElementUpdate(), or somewhere else? 2151899 9 ahmad.saleem792 2025-10-16 17:39:30 -0700 It works on Safari Technology Preview 230 (Tahoe) now. 2151955 10 graouts 2025-10-16 23:09:57 -0700 Nice! Must be a recent progression since this fails in Safari 26.0.1. 2151961 11 graouts 2025-10-16 23:43:39 -0700 Somewhat surprisingly, this progressed with 300706@main, the fix for bug 296048. 465812 2023-04-07 10:32:24 -0700 2023-04-07 10:32:24 -0700 Test bug-255114.html text/html 608 graouts PHN0eWxlPgoKLndhdmUgewogICAgd2lkdGg6IDEwMCU7CiAgICBoZWlnaHQ6IDEwMHB4Owp9Cgoj cGFnZS1kaXZpZGUgewogICAgYW5pbWF0aW9uOiBtb3ZlZm9yd2FyZCAxMHMgbGluZWFyIGluZmlu aXRlOwp9CgpAa2V5ZnJhbWVzIG1vdmVmb3J3YXJkIHsKICAgIGZyb20geyB0cmFuc2Zvcm06dHJh bnNsYXRlKC05MHB4KSB9CiAgICB0byAgIHsgdHJhbnNmb3JtOnRyYW5zbGF0ZSg4NXB4KSB9Cn0K Cjwvc3R5bGU+Cgo8c3ZnIGNsYXNzPSJ3YXZlIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAw MC9zdmciIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiB2aWV3Qm94 PSIwIDI0IDE1MCAyOCAiIHByZXNlcnZlQXNwZWN0UmF0aW89Im5vbmUiPgogICAgPGRlZnM+CiAg ICAgICAgPHBhdGggaWQ9InBhZ2UtZGl2aWRlIiB4PSI1MCIgZD0iTS0xNjAgNDRjMzAgMCA1OC0x OCA4OC0xOHM1OCAxOCA4OCAxOCA1OC0xOCA4OC0xOCA1OCAxOCA4OCAxOCB2NDRoLTM1MnoiPjwv cGF0aD4KICAgIDwvZGVmcz4KICAgIDx1c2UgeGxpbms6aHJlZj0iI3BhZ2UtZGl2aWRlIiB4PSI1 MCIgZmlsbD0idmFyKC0tcGFnZS1iZykiPjwvdXNlPgo8L3N2Zz4=