Closed Bug 1249497 Opened 4 years ago Closed 4 years ago

Make bdo element use unicode-bidi: isolate-override

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: xidorn, Assigned: xidorn)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

In bug 1218706, we applied "unicode-bidi: isolate" too all elements with dir attribute except bdo.

bdo needs "isolate-override", which we haven't fully supported yet, but once we finish that, we should change its default value.
What's missing in isolate-override support? I added it (prefixed) in bug 774335, and it's already used in html.css for bdo[dir="auto"]
Ah, I didn't notice it is already there for bdo[dir="auto"]. Then I guess we should make it the default for "bdo, bdo[dir]" directly.

The issue is that, we are still failing tests of isolate-override from CSSWG. It can be seen here: [1]. I compiled a build with unprefixed unicode-bidi values, and ran through the test suite, so the tests in this list which has one "passed" but lots of "failed" are likely due to that.

The tests with absolutely no "passed" are the problems we have. So it's bidi-isolate-override-005 ~ 010 for isolate-override.


[1] https://test.csswg.org/harness/results/css-writing-modes-3_dev/grouped/section/2/
Summary: Make unicode-bidi: isolate-override to bdo by default → Make bdo element use unicode-bidi: isolate-override
Attachment #8767084 - Flags: review?(jfkthame) → review+
Assignee: nobody → xidorn+moz
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d42940726072
Use isolate-override for bdo elements. r=jfkthame
https://hg.mozilla.org/mozilla-central/rev/d42940726072
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Added an info in:
https://developer.mozilla.org/en-US/Firefox/Releases/50#HTML

I don't think it is worth a mention on the element page itself.
You need to log in before you can comment on or make changes to this bug.