Closed Bug 1169267 Opened 6 years ago Closed 6 years ago

Directionality of dir=auto element is not always updated correctly when adding strong RTL characters to an element previously containing all neutrals

Categories

(Core :: Layout: Text and Fonts, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: jfkthame, Assigned: smontagu)

References

Details

Attachments

(2 files)

+++ This bug was initially created as a clone of Bug #1103348 +++

Modified STR to demonstrate bad behavior even after bug 1103348:

(1) In a new tab, load a simple dir=auto testcase:

  data:text/html,<div dir="ltr"><div contenteditable dir="auto"
    style="border:1px solid;padding:2px;width:200px;height:2em"></div></div>

(2) Click in the contenteditable <div>, and type some Western digits (e.g. "123"). The text is displayed with LTR directionality, as expected.

(3) Switch to an Arabic keyboard and append a few letters. Directionality correctly becomes RTL.

(4) Backspace to remove the RTL letters just added; when the last one is removed, the directionality correctly reverts to LTR.

(5) Now type the Arabic letters again. This time, the directionality incorrectly remains LTR.
(Removing 'regression' keyword, as I'm not sure this has ever worked properly.)
Keywords: regression
Assignee: nobody → smontagu
Attached patch PatchSplinter Review
Attachment #8613435 - Flags: review?(jfkthame)
Attached patch ReftestsSplinter Review
Attachment #8613436 - Flags: review?(jfkthame)
Attachment #8613435 - Flags: review?(jfkthame) → review+
Attachment #8613436 - Flags: review?(jfkthame) → review+
https://hg.mozilla.org/mozilla-central/rev/b89b70fcd8f2
https://hg.mozilla.org/mozilla-central/rev/fc7d84d79430
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.