Add direction:ltr to mathml.css

RESOLVED FIXED

Status

()

RESOLVED FIXED
15 years ago
15 years ago

People

(Reporter: rbs, Assigned: rbs)

Tracking

Trunk
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

857 bytes, patch
bzbarsky
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

15 years ago
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?]
(Assignee)

Comment 1

15 years ago
Created attachment 131878 [details]
patch
(Assignee)

Updated

15 years ago
Attachment #131878 - Attachment is obsolete: true
Attachment #131878 - Attachment is patch: false
(Assignee)

Comment 2

15 years ago
Created attachment 131879 [details] [diff] [review]
true patch
(Assignee)

Updated

15 years ago
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.
(Assignee)

Comment 5

15 years ago
Checked in.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

15 years ago
>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.