Closed Bug 732834 (mathjax-in-math-text) Opened 13 years ago Closed 13 years ago

Using MathJax fonts in mathematical text

Categories

(Core :: MathML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla13

People

(Reporter: fredw, Assigned: fredw)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(4 files, 2 obsolete files)

Attached patch Patch for the demo pages (obsolete) (deleted) — Splinter Review
I prepared new patches to fix this bug and sent them to the try server. For the moment, here is a patch for the website.
Assignee: nobody → fred.wang
Status: NEW → ASSIGNED
https://tbpl.mozilla.org/?tree=Try&rev=57febf443775 There are still issues with layout/reftests/bugs/355548-3.xml layout/reftests/bugs/355548-4.xml I don't know exactly which font-family value should apply here. The tests are the obsolete fontfamily attribute and the style attribute and I'm not sure how that all of this should interact.
Attached patch patch V1 (obsolete) (deleted) — Splinter Review
Attached patch patch V1 - part 2 (deleted) — Splinter Review
A note here on CSS, deprecated attributes and mathvariant: http://www.w3.org/TR/MathML3/chapter3.html#3.2.2.1 But I think here the problem is the rule added on mi.
Odd that the user-agent rule is overriding the author style="font-family:monospace;" but only when the fontfamily attribute is present.
Attachment #603475 - Attachment is obsolete: true
Attached patch patch V2 - part 1 (deleted) — Splinter Review
OK, Davide Cervone said that MathJax_Main is the same (more or less) as MathJax_Main italic. So I removed the rules on mi and that solves the problem of failing reftests.
Attachment #603623 - Attachment is obsolete: true
Attachment #604202 - Flags: review?(karlt)
Attachment #603624 - Flags: review?(karlt)
Attached patch Patch for the demo pages - V2 (deleted) — Splinter Review
And this patch is to use STIX/Asana Math/MathJax fonts on the demo pages.
Attachment #604203 - Flags: review?(karlt)
Alias: mathjax-in-math-text
Attachment #604202 - Flags: review?(karlt) → review+
Attachment #603624 - Flags: review?(karlt) → review+
Comment on attachment 604203 [details] [diff] [review] Patch for the demo pages - V2 r102707
Attachment #604203 - Flags: review?(karlt)
Attachment #604203 - Flags: review+
Attachment #604203 - Flags: checkin+
Attached patch patch - part 3 (deleted) — Splinter Review
Attachment #604899 - Flags: review?(karlt)
Attachment #604899 - Flags: review?(karlt) → review+
Attachment #603624 - Flags: checkin+
Attachment #604202 - Flags: checkin+
Keywords: checkin-needed
Whiteboard: [please push attachment 604899]
(In reply to Frédéric Wang from comment #12) > Created attachment 604899 [details] [diff] [review] > patch - part 3 This patch fixes a CSS rule. I'm not really concerned about this rule not being applied for now, but that cause error messages in the CSS console and users are likely to complain.
Docs got updated on the MathML font article and a note has been added to Firefox 13 for developers: https://developer.mozilla.org/en/Firefox_13_for_developers#MathML https://developer.mozilla.org/en/Mozilla_MathML_Project/Fonts Marking this as completed then.
Comment on attachment 604899 [details] [diff] [review] patch - part 3 https://hg.mozilla.org/integration/mozilla-inbound/rev/96e8cc4b2b20 I expect this correction should get merged before mozilla14.
Attachment #604899 - Flags: checkin+
Keywords: checkin-needed
Whiteboard: [please push attachment 604899]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: