104496 2012-12-09 14:41:43 -0800 SVG animations do not support dynamic condition targets 2026-01-29 12:36:04 -0800 1 1 1 Unclassified WebKit SVG 420+ Unspecified Unspecified NEW BrowserCompat, InRadar, LayerBasedSVGEngine P2 Normal --- 103811 1 pdr webkit-unassigned ahmad.saleem792 ap bfulgham fmalita graouts krit rniwa sabouhallawa schenney simon.fraser webkit-bug-importer zimmermann oldest_to_newest 786873 0 178440 pdr 2012-12-09 14:41:43 -0800 Created attachment 178440 Testcase Consider this example: <rect id="rectRed" width="100" height="100" x="0" y="0" fill="red"/> <rect id="rectGreen" width="100" height="100" x="100" y="0" fill="green"/> <set id="set" attributeName="x" to="0" xlink:href="#rectRed" begin="click"/> <script> var animation = document.getElementById('set'); animation.setAttributeNS('http://www.w3.org/1999/xlink', 'xlink:href', '#rectGreen'); </script> When we move the animation target from the red rect to the green rect, the click animation condition should move too but it does not. This currently works in Firefox and Opera. A related issue is detecting dynamic id changes such as if "blueRect.click" was used as the begin condition and blueRect's id was changed. One way to fix these issues would be to add condition targets to our target tracking infrastructure in SVGDocumentExtensions. 1881956 1 ahmad.saleem792 2022-07-10 11:37:22 -0700 I am able to reproduce this bug in Safari 15.5 on macOS 12.4 and Safari Technical Preview 148 using attached test case. As per instruction, upon clicking "green" box, does not animate over to red box. Although, it does for other browsers such as Chrome Canary 105 and Firefox Nightly 104. If I am testing it incorrectly, please retest accordingly. I tested using private window. Thanks! 1884334 2 bfulgham 2022-07-15 15:50:28 -0700 Works in Chrome and Firefox, fails in Safari. 1884335 3 webkit-bug-importer 2022-07-15 15:50:40 -0700 <rdar://problem/97098583> 2176300 4 ahmad.saleem792 2026-01-29 12:36:04 -0800 This fixes for me: ``` void SVGSMILElement::setTargetElement(SVGElement* target) { if (RefPtr timeContainer = m_timeContainer; timeContainer && hasValidAttributeName()) { if (RefPtr targetElement = m_targetElement.get()) timeContainer->unschedule(this, targetElement.get(), m_attributeName); if (target) timeContainer->schedule(this, target, m_attributeName); } // Check if we have event conditions that implicitly use the target element. bool hasImplicitTargetConditions = false; for (auto& condition : m_conditions) { if (condition.m_type == Condition::EventBase && condition.m_baseID.isEmpty()) { hasImplicitTargetConditions = true; break; } } if (RefPtr targetElement = m_targetElement.get()) { // Clear values that may depend on the previous target. stopAnimation(targetElement.get()); // Only disconnect/reconnect if we have conditions that depend on the target. if (hasImplicitTargetConditions) disconnectConditions(); } // If the animation state is not Inactive, always reset to a clear state before leaving the old target element. if (m_activeState != Inactive) endedActiveInterval(); m_targetElement = target; // Reconnect conditions if we have implicit target conditions if (hasImplicitTargetConditions && target && isConnected()) connectConditions(); } ``` 178440 2012-12-09 14:41:43 -0800 2012-12-09 14:41:43 -0800 Testcase events.html text/html 913 pdr PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8Ym9keT4KICBUZXN0IHRoYXQgYW5pbWF0aW9uIGNvbmRp dGlvbnMgYXJlIHRpZWQgdG8gdGhlIGFuaW1hdGlvbidzIHRhcmdldC48YnIvPgogIEluIHRoaXMg dGVzdCB3ZSBzd2l0Y2ggdGhlIGFuaW1hdGlvbiBmcm9tIHRoZSByZWQgYm94IHRvIHRoZSBncmVl biBib3guIENsaWNraW5nIHRoZSA8c3BhbiBzdHlsZT0iY29sb3I6Z3JlZW47IGZvbnQtd2VpZ2h0 OmJvbGQiPmdyZWVuPC9zcGFuPiBib3ggc2hvdWxkIG1vdmUgaXQgb3ZlciB0aGUgcmVkIGJveCBz byBubyByZWQgaXMgdmlzaWJsZS48YnIvPgo8c3ZnIHdpZHRoPSIzMDAiIGhlaWdodD0iMzAwIj4K PHJlY3QgaWQ9InJlY3RSZWQiIHdpZHRoPSIxMDAiIGhlaWdodD0iMTAwIiB4PSIwIiB5PSIwIiBm aWxsPSJyZWQiLz4KPHJlY3QgaWQ9ImNvdmVyUmVkUmVjdCIgd2lkdGg9IjEwMCIgaGVpZ2h0PSIx MDAiIHg9IjAiIHk9IjAiIGZpbGw9InRyYW5zcGFyZW50Ii8+PCEtLXByZXZlbnQgY2xpY2sgZXZl bnRzIGZyb20gcmVhY2hpbmcgI3JlZFJlY3QgLS0+CjxyZWN0IGlkPSJyZWN0R3JlZW4iIHdpZHRo PSIxMDAiIGhlaWdodD0iMTAwIiB4PSIxMDAiIHk9IjAiIGZpbGw9ImdyZWVuIi8+CjxzZXQgaWQ9 InNldCIgYXR0cmlidXRlTmFtZT0ieCIgdG89IjAiIHhsaW5rOmhyZWY9IiNyZWN0UmVkIiBiZWdp bj0iY2xpY2siLz4KPC9zdmc+CjxzY3JpcHQ+CndpbmRvdy5zZXRUaW1lb3V0KGZ1bmN0aW9uKCkg ewogIHZhciBhbmltYXRpb24gPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnc2V0Jyk7CiAgYW5p bWF0aW9uLnNldEF0dHJpYnV0ZU5TKCdodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rJywgJ3hs aW5rOmhyZWYnLCAnI3JlY3RHcmVlbicpOwp9LCAxKTsKPC9zY3JpcHQ+CjwvYm9keT4KPC9odG1s Pg==