232128 2021-10-21 19:51:08 -0700 Make transform a proper presentational attribute 2024-10-08 00:40:39 -0700 1 1 1 Unclassified WebKit SVG Safari Technology Preview Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=263712 InRadar P2 Normal --- 1 simon.fraser webkit-unassigned heycam karlcow maggotfish nmouchtaris sabouhallawa simon.fraser webkit-bug-importer zimmermann oldest_to_newest 1807501 0 simon.fraser 2021-10-21 19:51:08 -0700 Our SVG code currently does not treat `transform` as a presentational attribute. `transform-origin` is partially done (see the FIXME in createAttributeNameToCSSPropertyIDMap()). This all needs cleaning up. 1807503 1 heycam 2021-10-21 20:17:32 -0700 One tricky thing will be to make SVGTransformableElement.transform.{baseVal,animVal} do the right thing. https://drafts.csswg.org/css-transforms-1/#transform-attribute-dom 1809939 2 webkit-bug-importer 2021-10-28 19:52:16 -0700 <rdar://problem/84790424> 1993828 3 zimmermann 2023-11-20 00:07:14 -0800 Good catch Karl. Indeed this is the „lazy SVG attribute synchronization“ that apparently exposes this bug. We probably fail to unmap properly.. 1993830 4 zimmermann 2023-11-20 00:32:48 -0800 Oops ignore the last comment, wrong bug report. Notes that I tried this two years ago, but it was really hard to beat the speed of SVGTransformList parsing - even when adding multiple of fast paths for certain transformations as CSSParserFastPaths. But we should repeat the experiment. The only thing missing is rotation center support for rotate(). If there is interest, I can look up my code from a while ago