NEW278524
Replace CR/NL by space - don't remove altogether when xml:space=default 
https://bugs.webkit.org/show_bug.cgi?id=278524
Summary Replace CR/NL by space - don't remove altogether when xml:space=default
Ahmad Saleem
Reported 2024-08-22 09:02:46 PDT
Hi Team, While looking through Chrome bug tracker for SVG text failing test cases in Safari / WebKit, I stumbled upon another one. Test Case: https://jsfiddle.net/asyrzuz2/ ^ Chrome Canary 130 and Firefox Nightly 131 matches each other while Safari 18 Beta is outlier. Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/0c39a6f29e59594a7b9c412a8203af9fc43e4ecd WebKit Source: https://github.com/WebKit/WebKit/blob/b07168f663600ed8831b938973f80a80a0b5ca81/Source/WebCore/rendering/svg/RenderSVGInlineText.cpp#L50 Very easy to merge and can do quickly but I was thinking of first trying to land - https://github.com/WebKit/WebKit/pull/31081 but if it does not progress, at least I can try to do PR for this. Thanks!
Attachments
Add attachment proposed patch, testcase, etc.
Karl Dubost
Comment 1 2024-08-22 21:34:31 PDT
On Firefox and Chrome Foo bar Foo bar On Safari Foo bar Foobar <p> Foo bar </p> <svg> <text y="20"> Foo bar </text> </svg>
Radar WebKit Bug Importer
Comment 2 2024-08-22 21:34:44 PDT
<rdar://problem/134545958>
Ahmad Saleem
Comment 3 2024-08-29 07:04:15 PDT
Pull request: https://github.com/WebKit/WebKit/pull/32888
Ahmad Saleem
Comment 4 2025-06-19 12:06:16 PDT
It might lead to this regression - https://chromium-review.googlesource.com/c/chromium/src/+/664812
Note You need to log in before you can comment on or make changes to this bug.