Closed Bug 757703 Opened 12 years ago Closed 12 years ago

support unitless values for mpadded attributes

Categories

(Core :: MathML, defect)

Other Branch
x86_64
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: karlt, Assigned: fredw)

References

()

Details

(Keywords: dev-doc-complete, regression)

Attachments

(1 file, 1 obsolete file)

The regular expression for mpadded attributes in MathML 3.0 First Edition required units, but this was inconsistent with what was implied in the text.

The editor's draft for the Second Edition has now updated the regular expression to allow unitless values, which is what was intended and is consistent with the text.

http://lists.w3.org/Archives/Public/www-math/2012May/0027.html

Bug 677036 removed the special case for "0" in mpadded attributes, so that part of this bug is a regression.  Other unitless values have not been previously supported in Gecko.
Keywords: regression
I think we should really only support unitless "0" as in CSS:

http://www.w3.org/TR/css3-values/#lengths

See my comment here:
http://lists.w3.org/Archives/Public/www-math/2012May/0028.html
Summary: support unitless values for mapped attributes → support unitless values for mpadded attributes
Attached patch Patch V1 (obsolete) — Splinter Review
This patch should restore support for the "0" case and treats other unitless values as a multiple of the base size.
Assignee: nobody → fred.wang
Status: NEW → ASSIGNED
Attachment #626941 - Flags: review?(karlt)
Comment on attachment 626941 [details] [diff] [review]
Patch V1

>+      aCSSValue.SetPercentValue(floatValue);

It probably doesn't make any difference in practice, but it would be more consistent to use SetFloatValue(floatValue, eCSSUnit_Number).

I'm also happy if you only restore the "0" until the MathML WG makes a decision.
You made some good points against unitless values, but my guess is that they are unlikely to make a change if the intention was to allow unitless values.
Attachment #626941 - Flags: review?(karlt) → review+
Attached patch Patch V2Splinter Review
Attachment #626941 - Attachment is obsolete: true
Attachment #627186 - Flags: review?(karlt)
Keywords: dev-doc-needed
Attachment #627186 - Flags: review?(karlt) → review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/34af06cc57f6
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/34af06cc57f6
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
A note on this topic has been added to
https://developer.mozilla.org/en-US/docs/Firefox_15_for_developers#MathML

We probably don't need more docs here (see bug 677036#c30 ff).
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: