Closed Bug 1160455 Opened 5 years ago Closed 5 years ago

Add UI to configure the "fonts for mathematics" preferences

Categories

(SeaMonkey :: Preferences, enhancement)

enhancement
Not set

Tracking

(seamonkey2.38 fixed)

RESOLVED FIXED
seamonkey2.38
Tracking Status
seamonkey2.38 --- fixed

People

(Reporter: fredw, Assigned: fredw)

References

Details

Attachments

(1 file)

Bug 947654 will add font.*.x-math preferences options in order to configure the math fonts used for MathML. These should be configurable from the SeaMonkey Fonts preference menu.
See Also: → 1160456
Attached patch PatchSplinter Review
This seems to work for me.

Note that the x-math language is set on the <math> element for which mathml.css sets

  font-size: inherit;
  font-family: serif;

so really only font.name.*.x-math and minimum fonts have an effect.
(In reply to Frédéric Wang (:fredw) from comment #1)
> Created attachment 8600254 [details] [diff] [review]
> Patch
> 
> This seems to work for me.
> 
> Note that the x-math language is set on the <math> element for which
> mathml.css sets
> 
>   font-size: inherit;
>   font-family: serif;
> 
> so really only font.name.*.x-math and minimum fonts have an effect.

Frédéric, is this patch ready for review? If so please set the appropriate review flags.
Flags: needinfo?(fred.wang)
This depends on bug 947654, so I'll wait before asking review.
Flags: needinfo?(fred.wang)
Attachment #8600254 - Flags: review?(philip.chee)
Comment on attachment 8600254 [details] [diff] [review]
Patch

IanN is a better reviewer for Preference patches
Attachment #8600254 - Flags: review?(philip.chee) → review?(iann_bugzilla)
Attachment #8600254 - Flags: review?(iann_bugzilla) → review+
Keywords: checkin-needed
See Also: → 1170918
Can someone please commit the patch?
Flags: needinfo?(philip.chee)
a=me for CLOSED TREE
Status: NEW → ASSIGNED
Keywords: dev-doc-needed
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/a272b4bd0718
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: needinfo?(philip.chee)
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.38
Thanks Philip.
You need to log in before you can comment on or make changes to this bug.