Bug 585141 (mathml-screenshots)

Update Screenshot pages of Mozilla MathML Project

RESOLVED FIXED

Status

()

Core
MathML
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: fredw, Assigned: fredw)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

7 years ago
The page has moved to MDN:
https://developer.mozilla.org/en/Mozilla_MathML_Project/Mozilla_MathML_Screenshots

but we need to update the screenshots. We should check carefully if there is no regression since the time when the screenshots were taken.
(Assignee)

Updated

7 years ago
Alias: mathml-screenshots
(Assignee)

Comment 1

7 years ago
Created attachment 492795 [details]
Current screenshot of basics.xhtml

(with the patch of bug 427990 for XLink)

Compared with
http://www-archive.mozilla.org/projects/mathml/screenshots/basics.png

it seems that it remains only one regression: font-weight=bold is not inherited on single-char <mi/>'s. I guess it is the same issue as bug 449396.
(In reply to comment #1)
> it seems that it remains only one regression: font-weight=bold is not inherited
> on single-char <mi/>'s.

I don't think that's a bug.  IMO the default mathvariant on identifiers should override font-weight.  i.e. style should not affect identifiers.

> I guess it is the same issue as bug 449396.

That's a little different.  mathvariant actually changes (or, in the case of mstyle with bug 449396, should change) the characters for (i.e. meaning of) the identifiers.
(Assignee)

Comment 3

7 years ago
OK, that makes sense. Anyway, I'm not sure that the REC describes how style should be inherited from the parent of the <math/> element... So we should rather modify basics.xhtml to use mathvariant on <math/>. Similarly, we should replace the link by the recommended href attribute for MathML linking (I believed that the page contained a xlink but actually it is a HTML <a/>).
Depends on: 534968, 449396
Using mathvariant is not really right, because the page is trying to demonstrate "stylistic effects" available due to being in a browser, and mathvariant is not a stylistic effect.

Is there some other stylist effect that does not conflict with mathvariant but could be demonstrated instead?
Perhaps underline? or a rotation?
(Assignee)

Comment 5

7 years ago
OK, I see what you mean. I think underline currently adds some unwanted spaces under math frames (such that those on the link just before).
We can also simply modify the font-size.
font-size would be fine.
If text-shadow works, it could be a good candidate.
http://hacks.mozilla.org/2009/06/text-shadow/
(Assignee)

Comment 7

7 years ago
Created attachment 493120 [details] [diff] [review]
patch href + stylistic effects

text-shadow does not seem to work on the <mo/>'s.
(Assignee)

Updated

6 years ago
Attachment #493120 - Flags: review?(karlt)
Comment on attachment 493120 [details] [diff] [review]
patch href + stylistic effects

>-<a href="http://www.geocities.com/CapeCanaveral/Launchpad/3740/">
>-<math xmlns="&mathml;">
>+<math xmlns="&mathml;" href="http://en.wikipedia.org/wiki/Cape_Canaveral">

This was a link to information on Pythagoras' Theorem.
http://web.archive.org/web/20091027040032/http://geocities.com/CapeCanaveral/Launchpad/3740/
If you can find a page on Pythagoras' Theorem, that would be better.

The rest looks good, thanks.
(Assignee)

Comment 9

6 years ago
Created attachment 542223 [details] [diff] [review]
patch href + stylistic effects (V2)
Attachment #493120 - Attachment is obsolete: true
Attachment #493120 - Flags: review?(karlt)
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Attachment #542223 - Flags: review+
Whiteboard: [c-n svn]
Comment on attachment 542223 [details] [diff] [review]
patch href + stylistic effects (V2)

http://viewvc.svn.mozilla.org/vc?view=revision&revision=92371
Attachment #542223 - Flags: checkin+
Whiteboard: [c-n svn]
Keywords: checkin-needed
(Assignee)

Comment 11

6 years ago
In the equation "Y = a X + b" (after "write vector equations"), the use of mathvariant on mrow is not valid.

We could use a mstyle instead, but then the <mi>'s are no longer styled italic as in the original screenshot. Using style="font-weight: bold;" does not seem to work, the default style "italic" (not bold) is applied for the <mi>'s. For the moment, I can't see another option than setting the bold-italic style on each individual mi.
(In reply to comment #11)
> In the equation "Y = a X + b" (after "write vector equations"), the use of
> mathvariant on mrow is not valid.
> 
> We could use a mstyle instead, but then the <mi>'s are no longer styled
> italic as in the original screenshot.

Putting mathvariant on the the <math> element should be valid.  (I can't remember whether or not that works.)  I thought vector identifiers were usually not italic but mathvariant="bold-italic" should work fine.

> For the moment, I can't see another option than setting the
> bold-italic style on each individual mi.

mathvariant on the <math> element would apply to the operators also, and I don't think they should be bold (even though they are in the screenshot).

So mathvariant on each individual mi is probably appropriate.
(In reply to comment #12)
> mathvariant on the <math> element would apply to the operators also, and I
> don't think they should be bold (even though they are in the screenshot).

Oh, actually, there may be no bold variants of + and =, and so mathvariant on <math> might do the right thing.
(Assignee)

Updated

6 years ago
Assignee: nobody → fred.wang
(Assignee)

Comment 14

6 years ago
Created attachment 551269 [details] [diff] [review]
Move the mathvariant attribute from mrow to math

Let's make <mi>'s representing vector "bold".
Attachment #551269 - Flags: review?(karlt)
Attachment #551269 - Flags: review?(karlt) → review+
Comment on attachment 551269 [details] [diff] [review]
Move the mathvariant attribute from mrow to math

http://viewvc.svn.mozilla.org/vc?view=revision&revision=93440
Attachment #551269 - Flags: checkin+
(Assignee)

Updated

6 years ago
Blocks: 601872
(Assignee)

Comment 16

6 years ago
All the screenshots at

https://developer.mozilla.org/en/Mozilla_MathML_Project/Screenshots

are now updated, except the one for Emails.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.