WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
232128
Make transform a proper presentational attribute
https://bugs.webkit.org/show_bug.cgi?id=232128
Summary
Make transform a proper presentational attribute
Simon Fraser (smfr)
Reported
2021-10-21 19:51:08 PDT
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Cameron McCormack (:heycam)
Comment 1
2021-10-21 20:17:32 PDT
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
Radar WebKit Bug Importer
Comment 2
2021-10-28 19:52:16 PDT
<
rdar://problem/84790424
>
Nikolas Zimmermann
Comment 3
2023-11-20 00:07:14 PST
Good catch Karl. Indeed this is the „lazy SVG attribute synchronization“ that apparently exposes this bug. We probably fail to unmap properly..
Nikolas Zimmermann
Comment 4
2023-11-20 00:32:48 PST
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
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