Closed Bug 219928 Opened 21 years ago Closed 21 years ago

Add direction:ltr to mathml.css

Categories

(Core :: MathML, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: rbs, Assigned: rbs)

Details

Attachments

(1 file, 1 obsolete file)

857 bytes, patch
bzbarsky
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review
Spin-off from bug 192644.

Need to add this default CSS rule so tha bidi processing doesn't mess MathML:
math { 
  direction: ltr
}

[I wonder if unicode-bidi:embed|override|normal is relevant here too, smontagu?]
Attached file patch (obsolete) —
Attachment #131878 - Attachment is obsolete: true
Attachment #131878 - Attachment is patch: false
Attached patch true patchSplinter Review
Attachment #131879 - Flags: superreview?(bz-vacation)
Attachment #131879 - Flags: review?(bz-vacation)
Comment on attachment 131879 [details] [diff] [review]
true patch

r+sr=bzbarsky
Attachment #131879 - Flags: superreview?(bz-vacation)
Attachment #131879 - Flags: superreview+
Attachment #131879 - Flags: review?(bz-vacation)
Attachment #131879 - Flags: review+
In principle, if you have |display: inline|, you also need |unicode-bidi: embed|
according to http://www.w3.org/TR/REC-CSS2/visuren.html#propdef-direction

"For the 'direction' property to have any effect on inline-level elements, the
'unicode-bidi' property's value must be 'embed' or 'override'."

However, I understand from bug 192644 that adding |direction: ltr| worked, so I
am clearly missing something.
Checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
>However, I understand from bug 192644 that adding |direction: ltr| worked, so I
>am clearly missing something.

That may because that bug came from the <mtable>, which is internally a block,
and so it inherited the CSS rule from its (inline) math wrapper. However I added
"unicode-bidi: embed" as well, for conformity with the spec as you mentioned.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: