264184 2023-11-03 16:12:41 -0700 REGRESSION(264666@main): Layout of the <tspan> elements inside a <textPath> is incorrect 2023-12-13 11:41:29 -0800 1 1 1 Unclassified WebKit SVG WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=257160 InRadar P2 Normal --- 1 sabouhallawa zimmermann rbuis sabouhallawa simon.fraser webkit-bug-importer zimmermann oldest_to_newest 1989843 0 468478 sabouhallawa 2023-11-03 16:12:41 -0700 Created attachment 468478 test case Open the attached test case. Result: The layout of the text inside the circle is incorrect. This affects rendering of the MDN home page: https://developer.mozilla.org/en-US/. This is a regression of 264666@main which was the fix of bug 257160. 1989844 1 sabouhallawa 2023-11-03 16:13:16 -0700 rdar://117911214 1989850 2 zimmermann 2023-11-03 16:29:40 -0700 Thanks Said dir the reduction. I can have a look early next week… 1996680 3 zimmermann 2023-12-04 02:43:35 -0800 Working on it. 1996694 4 zimmermann 2023-12-04 04:44:12 -0800 Pull request: https://github.com/WebKit/WebKit/pull/21271 1996695 5 zimmermann 2023-12-04 04:46:00 -0800 Got a fix, and incorporated Saids testcase as new test in the PR. However I failed to make a proper reftest for this, as HTML/CSS is missing the primitives to layout text on path, and it's hard to simulate it in a cross-platform way, unless resorting to e.g. Ahem font etc. -- I didn't have the time to investigate, and also don't want to hold back the regression fix any longer. 1999346 6 ews-feeder 2023-12-13 11:41:26 -0800 Committed 271996@main (79238bc8e2d6): <https://commits.webkit.org/271996@main> Reviewed commits have been landed. Closing PR #21271 and removing active labels. 468478 2023-11-03 16:12:41 -0700 2023-11-03 16:12:41 -0700 test case mdn-home-page-modified.html text/html 778 sabouhallawa PHN0eWxlPgogICAgdGV4dFBhdGggewogICAgICAgIGZvbnQtc2l6ZTogMS41cmVtOwogICAgICAg IGZvbnQtd2VpZ2h0OiAzMDA7CiAgICB9Cjwvc3R5bGU+Cjxib2R5PgogICAgPHN2ZyB3aWR0aD0i Njc1IiBoZWlnaHQ9IjY3NSI+CiAgICAgICAgPHBhdGggZD0iTTMzNy41LDMzNy41IG0tMzIwLDAg YTMyMCwzMjAgMCAxLDEgNjQwLDAgYTMyMCwzMjAgMCAxLDEgLTY0MCwwIiBmaWxsPSJub25lIiBz dHJva2U9ImdyZWVuIiBpZD0iY2lyY2xlIj48L3BhdGg+CiAgICAgICAgPHRleHQgZHk9IjcwIiB0 ZXh0TGVuZ3RoPSIyMDEwIj4KICAgICAgICAgICAgPHRleHRQYXRoIHRleHRMZW5ndGg9IjIwMTAi IGhyZWY9IiNjaXJjbGUiPgogICAgICAgICAgICAgICAgJm5ic3A7Jm5ic3A7QQogICAgICAgICAg ICAgICAgPHRzcGFuPkI8L3RzcGFuPgogICAgICAgICAgICAgICAgPHRzcGFuPkM8L3RzcGFuPgog ICAgICAgICAgICAgICAgPHRzcGFuPkQ8L3RzcGFuPgogICAgICAgICAgICAgICAgPHRzcGFuPkU8 L3RzcGFuPgogICAgICAgICAgICAgICAgPHRzcGFuPkY8L3RzcGFuPgogICAgICAgICAgICAgICAg PHRzcGFuPkc8L3RzcGFuPgogICAgICAgICAgICAgICAgPHRzcGFuPkg8L3RzcGFuPgogICAgICAg ICAgICAgICAgPHRzcGFuPkk8L3RzcGFuPgogICAgICAgICAgICAgICAgPHRzcGFuPko8L3RzcGFu PgogICAgICAgICAgICAgICAgPHRzcGFuPks8L3RzcGFuPgogICAgICAgICAgICA8L3RleHRQYXRo PgogICAgICAgIDwvdGV4dD4KICAgIDwvc3ZnPgo8L2JvZHk+Cg==