Closed Bug 557481 Opened 10 years ago Closed 8 years ago

MathML start/demo pages: remove the alternate TeX and Mathematica stylesheets

Categories

(Core :: MathML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: fredw, Assigned: fredw)

References

Details

(Whiteboard: [inbound])

Attachments

(3 files)

The start pages as well as other demos provide some alternate stylesheets

<link rel="alternate stylesheet" title="TeX" href="tex.css" />
<link rel="alternate stylesheet" title="Mathematica" href="mathematica.css" />

This was initially aimed at comparing the rendering of different fonts. However, these two fonts are no longer recommended and are in unwanted categories 3 and 4 of bug 400938 comment 0 (personnaly, when I select View->Page Style->Tex, some symbols become wrong). When we support more Unicode fonts for stretching, it will be possible to add alternate stylesheets for STIX, Cambria, Asana...
Blocks: mathml-demos
No longer blocks: update-mathml-doc
I think it would be a good idea to wait support for Asana Math which is underway (bug 407439). Then we can replace Tex/Mathematica by STIX/Asana.

Other things to remember:

1) remove -moz-math-font-style-stretchy ("old: compatible with pre-1.4 builds"). I believe it is the only remaining issue that prevents to close bug 346222.

2) change the comments/examples about -moz-math-stretchy in mozilla-central.

find mathml/ | xargs grep "\-"moz"\-"math"\-"stretchy

mathfont.properties:# :-moz-math-stretchy in mathml.css.
mathml.css:   mo[myfonts="tex"]::-moz-math-stretchy {
mathml.css:   mo[myfonts="mathematica"]::-moz-math-stretchy {
mathml.css:   ::-moz-math-stretchy {
mathml.css:::-moz-math-stretchy {
nsMathMLChar.cpp:    stretchy fonts held by the leaf style context (from -moz-math-stretchy in
I don't know if the UI to choose stylesheets is still relevant since Firefox no longer have a menu bar by default and we have to use Alt + V. But at least, these patches remove the old TeX/Mathematica code.
Attachment #545219 - Flags: review?(karlt)
Attachment #545220 - Flags: review?(karlt)
Attachment #545219 - Flags: review?(karlt) → review+
Attachment #545220 - Flags: review?(karlt) → review+
Assignee: nobody → fred.wang
Comment on attachment 545220 [details] [diff] [review]
Change comments about TeX/Mathematica fonts to STIX/Asana fonts (svn)

http://viewvc.svn.mozilla.org/vc?view=revision&revision=93366
Attachment #545220 - Flags: checkin+
It seems that I forgot the changes that were outside the demo/ dir.
Attachment #551268 - Flags: review?(karlt)
Attachment #551268 - Flags: review?(karlt) → review+
Comment on attachment 551268 [details] [diff] [review]
Change comments about TeX/Mathematica fonts to STIX/Asana fonts (svn, part 2)

http://viewvc.svn.mozilla.org/vc?view=revision&revision=93439
Attachment #551268 - Flags: checkin+
Keywords: checkin-needed
Whiteboard: [inbound]
Comment on attachment 545219 [details] [diff] [review]
Change comments about TeX/Mathematica fonts to STIX/Asana fonts (mercurial)

http://hg.mozilla.org/integration/mozilla-inbound/rev/163605e2e60a
Attachment #545219 - Flags: checkin+
http://hg.mozilla.org/mozilla-central/rev/163605e2e60a
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.