Closed
Bug 696647
Opened 13 years ago
Closed 13 years ago
Remove unused (invalid) MathML atoms
Categories
(Core :: MathML, defect)
Core
MathML
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: fs, Assigned: fs)
References
Details
Attachments
(1 file)
(deleted),
patch
|
karlt
:
review+
|
Details | Diff | Splinter Review |
(Follow-up from bug 324472 comment 10)
Some MathML atoms in nsGkAtomList.h and nsTreeSanitizer.cpp seem to be unused (and wrongly mentioned in the MathML3 element index). We should probably remove them.
To be under consideration for now:
[1] malign: See http://lists.w3.org/Archives/Public/www-math/2011Sep/0001.html
[2] malignscope
"*Future* versions of MathML may provide an malignscope element that allows an alignment scope to be created around any MathML element [...]" (http://www.w3.org/TR/MathML3/chapter3.html#id.3.5.5.1) That is, malignscope isn't specified yet?
[3] mfraction
"[...] the non-MathML element mfraction (presumably in place of the MathML element mfrac) [...]" (http://www.w3.org/TR/MathML3/chapter3.html#id.3.3.5.3)
So "mfraction" does not exist and is just a sample for wrong markup ?
[4] monospaced
The MathML3 spec index links to http://www.w3.org/TR/MathML3/chapter2.html#fund.attval but there is no "monospaced" element/attribute ?!?
Comment 1•13 years ago
|
||
It would be a good idea to report these errors or questions to the MathML WG.
Assignee | ||
Comment 2•13 years ago
|
||
http://lists.w3.org/Archives/Public/www-math/2011Dec/0003.html
http://lists.w3.org/Archives/Public/www-math/2012Jan/0000.html
I can prepare a patch for removing these bits.
Assignee: nobody → elchi3
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•13 years ago
|
||
Attachment #586090 -
Flags: review?(karlt)
Updated•13 years ago
|
Blocks: namedspaceOverriding
Updated•13 years ago
|
Attachment #586090 -
Flags: review?(karlt) → review+
Comment 4•13 years ago
|
||
I tested this patch there:
https://tbpl.mozilla.org/?tree=Try&rev=bf7895f8e2fb
Keywords: checkin-needed
Comment 5•13 years ago
|
||
Keywords: checkin-needed
Target Milestone: --- → mozilla12
Comment 6•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•