Closed Bug 717587 Opened 8 years ago Closed 8 years ago

Need to support new MathML3 href attribute

Categories

(SeaMonkey :: UI Design, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

References

Details

Attachments

(1 file, 1 obsolete file)

As of bug 427990 MathML nodes can now use an href attribute instead of an XLink to create a link inside a mathematical formula.

Some test cases are available on bug 534968.
Attached patch Possible patch (obsolete) — Splinter Review
Fixes, so far:
Ctrl+Click on MathML Link to open in new tab
Correct menuitems on right-click of MathML link
Properties menuitem works on MathML link
Page Info detects MathML links
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #588019 - Flags: review?(iann_bugzilla)
Comment on attachment 588019 [details] [diff] [review]
Possible patch

This looks good so far but was just wondering if there was any code duplication that could be removed around the metadata.js and pageInfo.js changes.
Attachment #588019 - Flags: review?(iann_bugzilla) → review+
Attachment #588019 - Attachment is obsolete: true
Attachment #588429 - Flags: review?(iann_bugzilla)
Attachment #588429 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 7a34e038ee94 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
I'm 99% sure this caused bug 720225. :-(
Blocks: 720225
Now that bug 720225 is fixed, I get:

Error: linkNode.hasAttributeNS is not a function
Source File: chrome://communicator/content/contentAreaClick.js
Line: 76

when clicking an empty space of a website in a browser window. [I guess this can be reproduced in many other ways.] When the error occurs for me, linkNode.nodeName = "#document".
Ah yes, I lost the element check that the previous code had. File a bug?
You need to log in before you can comment on or make changes to this bug.