New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 865502 link

Starred by 2 users
Status: Available
Owner: ----
Cc:
█ krajshree@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug


Show other hotlists

Hotlists containing this issue:
Watched


Sign in to add a comment

In an RTL setting, brackets around LTR Text will get flipped and moved to the other side

Reported by n...@twitter.com, Jul 19 
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36

Steps to reproduce the problem:
1. Set a paragraph to direction='rtl' using css
2. In the <p>, type something like 'TEST(HELLO)' 
3. See that it prints (TEST(HELLO

What is the expected behavior?
Expects TEST(HELLO)

What went wrong?
Chrome doesn't balance parenthesis according to the new bidi algorithm

See - http://w3c.github.io/i18n-drafts/articles/inline-bidi-markup/index.en#mirrored 

Did this work before? N/A 

Does this work in other browsers? No
 Safari has this problem too.

See - http://w3c.github.io/i18n-drafts/articles/inline-bidi-markup/index.en#mirrored. 

Chrome version: 67.0.3396.99  Channel: stable
OS Version: OS X 10.13.6
Flash Version:

Comment 1 by dtapu...@chromium.org, Jul 19

Components: Blink>Layout

Comment 2 by vamshi.kommuri@chromium.org, Jul 20

Labels: Needs-Triage-M67

Comment 3 by krajshree@chromium.org, Jul 20

Cc: krajshree@chromium.org
Labels: Needs-Feedback Triaged-ET
nli@ - Thanks for filing the issue...!!

Could you please provide a sample test file/url to test the issue from TE-end. This will help us in triaging the issue further.

Thanks...!!

Comment 4 by n...@twitter.com, Jul 20 

The second line uses a left-to-right mark, which fixes the issue, but from the link you can see that it's supposed to fix it on its own, without needing the ltr mark.
test.html
178 bytes View Download

Comment 5 by n...@twitter.com, Jul 20 

Thank you! Let me know if you need more information.
Project Member

Comment 6 by sheriffbot@chromium.org, Jul 20

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by e...@chromium.org, Jul 23

Labels: Fixed-In-LayoutNG
Status: Available (was: Unconfirmed)

Comment 8 by e...@chromium.org, Jul 23 

Thanks for the report and the test case!
The good news is that we've already fixed this in the new layout engine. The bad news is that it'll be a little while longer before that is released.
Keeping the bug open until then.

Comment 9 by n...@twitter.com, Jul 24 

Thank you!
►

Sign in to add a comment