746987 - unicode-bidi:plaintext is supposed to work on inline elements

Status: RESOLVED FIXED

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

Description

[Aharon (Vladimir) Lanin]

When unicode-bidi:-moz-plaintext was originally implemented, the CSS Writing Modes Level 3 spec said that the plaintext value was not supposed to have any effect on inline elements. However, that changed some time ago. It currently states as follows:

This value behaves as ‘isolate’ except that for the purposes of the Unicode bidirectional algorithm, the base directionality of each bidi paragraph immediately contained by the element is determined not by the element's computed ‘direction’ as usual, but by following the heuristic in rules P2 and P3 of the Unicode bidirectional algorithm. An element immediately contains a bidi paragraph if the element itself, but none of its descendants, both contains the entire bidi paragraph and is either a block container or a bidi-isolating inline.

In other words, plaintext is supposed to bidi-isolate inline elements just like unicode-bidi:isolate, while giving each bidi paragraph bounded by the element its own base direction by the usual plaintext algorithm (just like it does for non-inline elements).

Comment 1

[Aharon (Vladimir) Lanin]

Attachment: Test case 1

Comment 2

[Simon Montagu :smontagu]

Attachment: Patch

Comment 3

[Simon Montagu :smontagu]

Attachment: The test as reftest

Comment 4

[Simon Montagu :smontagu]

Attachment: Patch

The right diff this time

Comment 5

[Aharon (Vladimir) Lanin]

Attachment: Test case 2

Comment 6

[Aharon (Vladimir) Lanin]

Attachment: Test case 3

Comment 7

[Aharon (Vladimir) Lanin]

Attachment: Test case 4

Comment 8

[Aharon (Vladimir) Lanin]

BTW, plaintext is only supposed to affect alignment when it is on a containing block.

Comment 9

[Simon Montagu :smontagu]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e48d32ee1a5b
https://hg.mozilla.org/integration/mozilla-inbound/rev/15a028a333d2

Comment 10

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e48d32ee1a5b
https://hg.mozilla.org/mozilla-central/rev/15a028a333d2

Comment 11

[Jean-Yves Perrier [:teoli]]

I've added a note in:
https://developer.mozilla.org/en-US/docs/CSS/unicode-bidi
https://developer.mozilla.org/en-US/docs/Firefox_15_for_developers