Bug 123846 - unicode-bidi: plaintext does not work properly when a line wraps
Summary: unicode-bidi: plaintext does not work properly when a line wraps
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks:
 
Reported: 2013-11-05 18:43 PST by Ryosuke Niwa
Modified: 2022-09-30 17:26 PDT (History)
9 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-11-05 18:43:39 PST
Merge https://chromium.googlesource.com/chromium/blink/+/d5088d76fcf75eaf23e30be43ebf119b2343dfed

unicode-bidi: plaintext never worked properly when there is a line wrapping in a 
span with unicode-bidi: plaintext.
The problem happens because constructBidiRunsForSegment does not have
 enough logic to understand the difference between line wrapping vs line
 breaking, the current patch just adds the necessary logic(e.g when a text has a
 line wrapping it needs to determine the directionality of the paragraph from the beginning of the isolatedInline)
Comment 1 Ahmad Saleem 2022-09-30 17:12:24 PDT
I took test case from the patch:

Link (removed -webkit-) - https://jsfiddle.net/q8m3o1tu/show
Link (not removed -webkit-) - https://jsfiddle.net/t9r2s41n/1/show

All browsers (Chrome Canary 108, Firefox Nightly 107 and Safari 16) render all boxes same with the same direction of arrows and stuff (no wrapping problem).

Do we need this anymore? Thanks!
Comment 2 zalan 2022-09-30 17:26:05 PDT
There's also a hidden IFC progression here.