264752 2023-11-13 10:03:53 -0800 Negative SVGTransform scale values should be correctly stringified 2024-01-29 19:51:10 -0800 1 1 1 Unclassified WebKit SVG WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=267401 BrowserCompat, InRadar P2 Normal --- 1 ahmad.saleem792 webkit-unassigned karlcow sabouhallawa webkit-bug-importer zimmermann oldest_to_newest 1992152 0 ahmad.saleem792 2023-11-13 10:03:53 -0800 Hi Team, While going through Blink's commit, I came across another failing test case: Test Case: https://jsfiddle.net/7nqg2L01/show ^ Firefox Nightly 121: FAIL g.getAttribute('transform') should be scale(-2 -4). Was scale(-2, -4). ^ Chrome Canary 121: PASS g.getAttribute('transform') is "scale(-2 -4)" ^ Safari Technology Preview 182 (Don't show negative): FAIL g.getAttribute('transform') should be scale(-2 -4). Was scale(2 4). ___ Blink Commit: https://chromium.googlesource.com/chromium/blink/+/8a7252a2602bd8067c83d48c8fd525ef322708c2 Just wanted to raise, so we can fix it. Thanks! 1993945 1 webkit-bug-importer 2023-11-20 10:04:14 -0800 <rdar://problem/118656892> 1994038 2 karlcow 2023-11-20 18:08:04 -0800 There are potentially two different issues * negative to positive number * serialization with/without comma as the 3 browsers return something different. IN OUT Safari: scale(-2, -4) scale(2 4). No comma, positive numbers Firefox: scale(-2, -4) scale(-2, -4) Comma, negative numbers Chrome: scale(-2, -4) scale(-2 -4) No comma, negative numbers I have the feeling that Firefox is right here for the comma. https://drafts.csswg.org/css-transforms/#funcdef-transform-scale I didn't find a reference why it should be serialize differently, aka without the comma. But WebKit definitely needs to return negative numbers. 1994039 3 karlcow 2023-11-20 18:09:47 -0800 The test: var svg = document.createElementNS('http://www.w3.org/2000/svg', 'svg'); var g = document.createElementNS('http://www.w3.org/2000/svg', 'g'); var transform = svg.createSVGTransform(); transform.setScale(-2, -4); g.transform.baseVal.appendItem(transform) g.getAttribute('transform') ​ 2003427 4 ahmad.saleem792 2024-01-06 17:51:27 -0800 We need to update this: https://searchfox.org/wubkat/rev/c22ca98693565713efde37e8311190dd00e165ef/Source/WebCore/svg/SVGTransformValue.h#267 void appendScale(StringBuilder& builder) const { appendFixedPrecisionNumbers(builder, m_matrix->value().xScale(), m_matrix->value().yScale()); } TO: void appendScale(StringBuilder& builder) const { appendFixedPrecisionNumbers(builder, m_matrix->a(), m_matrix->d()); } __ Compiles and make us pass but match 'Chrome'. As for ',' issue highlighted in Comment 02. I think it might be 'Parser' issue? I am happy to fix this and then have separate bug to discuss ',' issue. 2003428 5 ahmad.saleem792 2024-01-06 18:03:16 -0800 Draft PR - https://github.com/WebKit/WebKit/pull/22475 To see if it fails anything else on EWS. :-) 2004324 6 ews-feeder 2024-01-10 21:17:24 -0800 Committed 272885@main (2efdea0ba354): <https://commits.webkit.org/272885@main> Reviewed commits have been landed. Closing PR #22475 and removing active labels.