Closed Bug 700433 Opened 13 years ago Closed 12 years ago

MathML demo pages: inclusion of HTML fragments in MathML is not valid

Categories

(www.mozilla.org :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: fredw, Assigned: fredw)

References

Details

Attachments

(2 files, 2 obsolete files)

Follow-up of bug 445029 comment 113

Our demos contain HTML fragments inside MathML code but these inclusions are not valid. This does not seem to be defined anywhere in W3C specs.
Attached patch patch (obsolete) — Splinter Review
demo/basics.html:

- I replaced <img>'s in MathML by <mspace>'s with background-image
- I replaced resizable <input>'s in MathML by <mspace>'s

demo/extras.html:

- I replaced <span>'s with mouseover attributes in MathML by <mtext>'s with an event attached using addEventListener.
- I replaced the insertion of XHTML in MathML (which is AFAIK not defined in any spec) by an animated SVG image with MathML elements inside <foreignObject>'s.

The remaining errors are for the "title" attributes on MathML elements (bug 700440).
Assignee: nobody → fred.wang
Status: NEW → ASSIGNED
Attachment #598958 - Flags: review?(karlt)
Comment on attachment 598958 [details] [diff] [review]
patch

I checked in the changes to extras.html, thanks.
http://viewvc.svn.mozilla.org/vc?view=revision&revision=102017

Re basics.html:

> <mi>bongo</mi>
> <mo>=</mo>
> <mfrac>
>-  <img width="197" height="200" src="http://www.mozilla.org/images/logo-star.gif" alt="logo-star.gif"/>
>+  <mspace style="background-image: url('http://www.mozilla.org/images/logo-star.gif');" width="197px" height="200px"></mspace>

I guess the text "such as mixing MathML with other markups" should be removed
now as this is using CSS rather than another markup.

>-Fill the gaps in this matrix with resizable input fields.
>+Fill the gaps in this matrix with resizable mspaces.

I think the idea here was that the input fields allowed entry of numbers to
complete the entries in the matrix.  It doesn't make so much sense to replace with mspaces.

Can we think of a completely different example using javascript and the DOM?
Perhaps changing the font size of the left and right hand sides of the
equation?  Or some way that mspaces change the layout of an equation?  If there is another example of DOM manipulation elsewhere, then we could just remove this.
Attachment #598958 - Flags: review?(karlt)
Attached patch patch basicsSplinter Review
OK, I've updated the dynamic equation in basics.html. Now we can change mathsize.
I put the equation below, so that the +/- don't move when we change the size.
Attachment #599198 - Flags: review?(karlt)
Comment on attachment 599198 [details] [diff] [review]
patch basics

Looks good, thanks again.

http://viewvc.svn.mozilla.org/vc?view=revision&revision=102037
Attachment #599198 - Flags: review?(karlt) → review+
OK, I verified that all the projects/mathml pages validate, except demo/extras.html (bug 700440).
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
According to David Carlisle:

"MathML3 suggests that the token elements are extended in any compound
document type to allow the host markup.
HTML5 does this and most inline html is allowed inside <mtext>  so
<mfrac><mtext><svg>....</svg></mtext><mtext><img...</mtext></mfrac>
is valid html5 with html and svg in mathml."

So I guess some of the changes

<img> => <mspace>
<input> => <mspace>

can be reverted and we can demoonstrate inclusion of SVG/HTML in MathML in demo/extras.html
Attachment #598958 - Attachment is obsolete: true
Attached patch Mixing MathML, SVG, HTML (obsolete) — Splinter Review
Attachment #638088 - Flags: review?(karlt)
Apparently, there is also a typo: animateTransform@keyTime should be animateTransform@keyTimes (with a s).
Attachment #638088 - Attachment is obsolete: true
Attachment #638088 - Flags: review?(karlt)
Attachment #638128 - Flags: review?(karlt)
Comment on attachment 638128 [details] [diff] [review]
Mixing MathML, SVG, HTML - v2

r107300
Attachment #638128 - Flags: review?(karlt) → review+
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: