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)
Core
MathML
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)
(deleted),
patch
|
karlt
:
review+
fredw
:
checkin+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
karlt
:
review+
fredw
:
checkin+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
karlt
:
review+
karlt
:
checkin+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
karlt
:
review+
karlt
:
checkin+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•13 years ago
|
||
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
Assignee | ||
Comment 2•13 years ago
|
||
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.
Assignee | ||
Comment 3•13 years ago
|
||
Assignee | ||
Comment 4•13 years ago
|
||
Assignee | ||
Comment 5•13 years ago
|
||
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.
Comment 6•13 years ago
|
||
Odd that the user-agent rule is overriding the author style="font-family:monospace;" but only when the fontfamily attribute is present.
Assignee | ||
Updated•13 years ago
|
Attachment #603475 -
Attachment is obsolete: true
Assignee | ||
Comment 7•13 years ago
|
||
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)
Assignee | ||
Updated•13 years ago
|
Attachment #603624 -
Flags: review?(karlt)
Assignee | ||
Comment 8•13 years ago
|
||
And this patch is to use STIX/Asana Math/MathJax fonts on the demo pages.
Attachment #604203 -
Flags: review?(karlt)
Assignee | ||
Updated•13 years ago
|
Alias: mathjax-in-math-text
Updated•13 years ago
|
Attachment #604202 -
Flags: review?(karlt) → review+
Updated•13 years ago
|
Attachment #603624 -
Flags: review?(karlt) → review+
Comment 9•13 years ago
|
||
Attachment #604203 -
Flags: review?(karlt)
Attachment #604203 -
Flags: review+
Attachment #604203 -
Flags: checkin+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed,
dev-doc-needed
Comment 10•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/53545c1c626a
https://hg.mozilla.org/integration/mozilla-inbound/rev/5df18f54c148
Keywords: checkin-needed
Target Milestone: --- → mozilla13
Comment 11•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/53545c1c626a
https://hg.mozilla.org/mozilla-central/rev/5df18f54c148
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 12•13 years ago
|
||
Attachment #604899 -
Flags: review?(karlt)
Updated•13 years ago
|
Attachment #604899 -
Flags: review?(karlt) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #603624 -
Flags: checkin+
Assignee | ||
Updated•13 years ago
|
Attachment #604202 -
Flags: checkin+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Whiteboard: [please push attachment 604899]
Assignee | ||
Comment 13•13 years ago
|
||
(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.
Comment 14•13 years ago
|
||
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.
Keywords: dev-doc-needed → dev-doc-complete
Comment 15•13 years ago
|
||
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+
Updated•13 years ago
|
Keywords: checkin-needed
Whiteboard: [please push attachment 604899]
Comment 16•13 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•