Bug 246888

Summary: SVG text in RTL mixes characters
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: karlcow, mmaxfield, sabouhallawa, webkit-bug-importer, zimmermann
Priority: P2 Keywords: BrowserCompat, InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=78711
Attachments:
Description Flags
Patch Applied none

Ahmad Saleem
Reported 2022-10-21 15:42:43 PDT
Hi Team, I tried to search for this bug but couldn't find to do Blink commit merge, hence, I am creating this as new: Test Case failing in Safari 16 and STP 155 - https://jsfiddle.net/L3fmwvre/show Chromium / Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=167777 Where to merge - https://github.com/WebKit/WebKit/blob/ce0ee18aa01d4d38b4dc7e9535ea61e24234c6cb/Source/WebCore/rendering/svg/SVGRootInlineBox.cpp#L262 _____ Just wanted to create a bug for it. Thanks!
Attachments
Patch Applied (219.58 KB, image/png)
2023-03-06 12:33 PST, Ahmad Saleem 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Karl Dubost
Comment 1 2022-10-24 18:32:23 PDT
fwiw I tempted to apply the same code change to WebKit. and the SVG doesn't display anymore. So there's probably more to it.
Ahmad Saleem
Comment 2 2022-10-25 00:43:51 PDT
(In reply to Karl Dubost from comment #1) > fwiw I tempted to apply the same code change to WebKit. > and the SVG doesn't display anymore. So there's probably more to it. Yes - it requires prerequisite, I noticed in my PR as well. https://github.com/WebKit/WebKit/pull/5656#issuecomment-1288157540
Radar WebKit Bug Importer
Comment 3 2022-10-28 15:43:22 PDT
<rdar://problem/101695671>
Ahmad Saleem
Comment 4 2023-03-06 12:33:00 PST
Created attachment 465321 [details] Patch Applied It does not fix completely but make it less broken as can be seen from attached picture. Thanks!
EWS
Comment 5 2023-03-10 05:02:51 PST
Committed 261495@main (46f0f6ec886e): <https://commits.webkit.org/261495@main> Reviewed commits have been landed. Closing PR #11194 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.