Closed Bug 601436 Opened 9 years ago Closed 8 years ago

mtable 'align' attribute: negative rownumbers are wrongly interpreted

Categories

(Core :: MathML, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: distler, Assigned: fredw)

References

()

Details

(Keywords: dev-doc-complete)

Attachments

(4 files)

User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b7pre) Gecko/20100926 Firefox/ SeaMonkey/2.1b1pre
Build Identifier: 

According to MathML Spec, the mtable 'align' attribute can accept a positive or negative integer, indicating the row of the table, to which the alignment should be applied. 1,2,... indicate the first, second, ... row. -1, -2, ... indicate the last, next-to-last, ... row.

Gecko recognizes positive values, but not negative ones.

See the testcase. 

Reproducible: Always
Attached file testcase
Attached image incorrect rendering
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch PatchSplinter Review
Actually, negative values are taken into account but wrongly interpreted: -1 is considered the penultimate row, -2 the antepenultimate row etc
Assignee: nobody → fred.wang
Attachment #551305 - Flags: review?(roc)
Summary: mtable 'align' attribute does not accept negative rownumbers → mtable 'align' attribute: negative rownumbers are wrongly interpreted
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/1cfc6b117e15
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
(In reply to Florian Scholz [:fs] from comment #6)
> https://developer.mozilla.org/en/Firefox_8_for_developers#MathML
> https://developer.mozilla.org/en/MathML/Element/mtable#attr-align

Ah.  I was originally going to comment here that dev-doc-needed did not seem to apply to fixing bugs, but I see what was really being asked for was updating developers section of release notes to indicate the bug was fixed.  Perhaps we need a different keyword like dev-relnote-needed to make this more clear.
Or perhaps it really should have just had the relnote keyword instead of devdoscs-needed.  Devdocs-needed would seem to imply that the developers wiki needs to be updated.
That's exactly what it means. relnotes are user-facing; dev-doc is developer-facing.
Appears to be broken again.
(In reply to distler from comment #10)
> Created attachment 8819379 [details]
> Incorrect rendering in latest Firefox
> 
> Appears to be broken again.

Can you please open a new bug?
It would also help to find a regression window: https://mozilla.github.io/mozregression/quickstart.html
You need to log in before you can comment on or make changes to this bug.