mtable 'align' attribute: negative rownumbers are wrongly interpreted

RESOLVED FIXED in mozilla8

Status

()

Core
MathML
RESOLVED FIXED
7 years ago
4 months ago

People

(Reporter: distler, Assigned: fredw)

Tracking

({dev-doc-complete})

unspecified
mozilla8
x86
Mac OS X
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(4 attachments)

(Reporter)

Description

7 years ago
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
(Reporter)

Comment 1

7 years ago
Created attachment 480482 [details]
testcase
(Reporter)

Comment 2

7 years ago
Created attachment 480484 [details]
incorrect rendering
(Assignee)

Updated

7 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 3

6 years ago
Created attachment 551305 [details] [diff] [review]
Patch

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)
(Assignee)

Updated

6 years ago
Summary: mtable 'align' attribute does not accept negative rownumbers → mtable 'align' attribute: negative rownumbers are wrongly interpreted
Attachment #551305 - Flags: review?(roc) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
http://hg.mozilla.org/integration/mozilla-inbound/rev/1cfc6b117e15
Keywords: checkin-needed
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/1cfc6b117e15
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8

Updated

6 years ago
Keywords: dev-doc-needed
https://developer.mozilla.org/en/Firefox_8_for_developers#MathML
https://developer.mozilla.org/en/MathML/Element/mtable#attr-align
Keywords: dev-doc-needed → dev-doc-complete
(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.
Keywords: dev-doc-complete → relnote
That's exactly what it means. relnotes are user-facing; dev-doc is developer-facing.
Keywords: relnote → dev-doc-complete
(Reporter)

Comment 10

4 months ago
Created attachment 8819379 [details]
Incorrect rendering in latest Firefox

Appears to be broken again.
(Assignee)

Comment 11

4 months ago
(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.