Closed Bug 448509 Opened 16 years ago Closed 15 years ago

Make MathML Torture Test (http://www.mozilla.org/projects/mathml/demo/texvsmml.xhtml) CSS validate

Categories

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

x86
Windows Vista
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: cop3252, Unassigned)

References

()

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.0.1) Gecko/2008070208 Firefox/3.0.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.0.1) Gecko/2008070208 Firefox/3.0.1

Bug 267882 made the above page valid XHTML but NOT valid CSS.
The page should use valid CSS and pass the W3C CSS validator at
http://jigsaw.w3.org/css-validator/

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
Product: mozilla.org → Websites
This bug might be fixed if the DTD was changed to be the one that the W3C recommends.  This document is using the very old 

<!DOCTYPE html
    PUBLIC "-//W3C//DTD XHTML 1.1 plus MathML 2.0//EN"
           "http://www.w3.org/TR/MathML2/dtd/xhtml-math11-f.dtd"

recommended in the old:

Mathematical Markup Language (MathML) Version 2.0
W3C Recommendation 21 February 2001

instead of using the latest: 

Mathematical Markup Language (MathML) Version 2.0 (Second Edition)
W3C Recommendation 21 October 2003

which recommends:

<!DOCTYPE html
    PUBLIC "-//W3C//DTD XHTML 1.1 plus MathML 2.0//EN"
           "http://www.w3.org/Math/DTD/mathml2/xhtml-math11-f.dtd"

Note that the location of the file xhtml-math11-f.dtd is in a new location.
Summary: Make http://www.mozilla.org/projects/mathml/demo/texvsmml.xhtml CSS validate → Make MathML Torture Test (http://www.mozilla.org/projects/mathml/demo/texvsmml.xhtml) CSS validate
Here is a patch that fixes the problem

It is my first patch.
I filed bug  http://www.w3.org/Bugs/Public/show_bug.cgi?id=6602
with the CSS-validator.  This but is now fixed.

This fix allows documents using the old DTD  
"http://www.w3.org/TR/MathML2/dtd/xhtml-math11-f.dtd"
to be validated.

This bug is now fixed.  

The patch I gave should still be applied 
since the DTD used is no longer the recommended one to use.

Bug http://www.w3.org/Bugs/Public/show_bug.cgi?id=6573
is fixed and will be pushed to the production HTML validator
and will allow a document using the recommended DTD to validate
without any warnings.
Status: UNCONFIRMED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Joe, it seems that your patch is still not applied. Did you ask for a review?

https://developer.mozilla.org/En/Developer_Guide/How_to_Submit_a_Patch
Comment on attachment 361809 [details] [diff] [review]
Patch to fix the problem

fixed Doctype
Attachment #361809 - Flags: review?(mozbugz)
Please update the Email address of Karl Tomlinson as listed at 
http://www.mozilla.org/about/owners.html#mathml
since is is rejected by the patch review request form.
I used all the examples given in the torture test and wrote a 
smaller page that suggest replace the torture page.  See:

http://eyeasme.com/Joe/MathML/MathML_browser_test.html
Attachment #361809 - Flags: review?(mozbugz) → review+
(In reply to comment #7)
Nice test page, thanks.
I'm hoping that with the HTML5 parser (for MathML in HTML) we'll be able to find a wiki solution for these pages and updates will be easier.
MathML works great with HTML5 uing the latest Firefox trunk builds 

The problem is with the validators.
The W3C validator and the Validator.nu (X)HTML5 Validator
both need to be updated first, since they do not parse HTML with MathML properly.
For example this page is HTML5 with MathML:

https://eyeasme.com/Joe/MathML/HTML5_MathML_browser_test.html
Comment on attachment 361809 [details] [diff] [review]
Patch to fix the problem

http://validator.w3.org/ is still warning as reported at http://www.w3.org/Bugs/Public/show_bug.cgi?id=6573 and saying "The recommended System Identifier for XHTML 1.1 plus MathML 2.0 is http://www.w3.org/TR/MathML2/dtd/xhtml-math11-f.dtd." so let's wait for the validator bug to be fixed before making this change.
So I guess we can take the patch now:
http://validator.w3.org/whatsnew.html#v086
Keywords: checkin-needed
Depends on: 550822
Comment on attachment 361809 [details] [diff] [review]
Patch to fix the problem

I checked this in, thank you:
http://viewvc.svn.mozilla.org/vc?view=revision&revision=63917

Sorry, I'm a novice with svn.
I used 'svn ci --username "Joe Java <cop3252@yahoo.com>"', but that doesn't
seem to have been reflected in the advertised authorship.
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

Creator:
Created:
Updated:
Size: