668618 - ContentAreaClick doesn't handle MathML 3 links with just the href attribute

Status: RESOLVED DUPLICATE of bug 534968

(Firefox :: General, defect)

Description

[Marco Bonardo [:mak]]

MathML 3 allows to set href attribute on any element, but this is not an XLink nor namespaces. We support XLinks but not these direct attributes, we never did.

Comment 1

[Marco Bonardo [:mak]]

A couple tests to add:

<math id="mathhref" xmlns="http://www.w3.org/1998/Math/MathML" href="http://mochi.test/moz/"><mtext>MathML href</mtext></math>

<math id="mathhref" xmlns="http://www.w3.org/1998/Math/MathML"><mtext href="http://mochi.test/moz/">MathML href</mtext></math>

Comment 2

[Frédéric Wang (:fredw)]

Actually, we do have some support for the MathML3 href now, the only thing missing is the menu items when you right click on a link.

See bug 534968 comment 4

Comment 3

[Marco Bonardo [:mak]]

yep, I know, but browser does not, yet.

Comment 4

[Bill Gianopoulos [:WG9s]]

(In reply to comment #3)
> yep, I know, but browser does not, yet.

But the point is that there is a pending patch on bug 534968 that fixes both the ctrl-click and the right click context menu issues.  It has bit-rotted a bit, but Fred has an updated version in his patch queue at https://raw.github.com/fred-wang/MozillaCentralPatches/master/mathml-href.diff

It might not be the ideal solution, but it does work, so should be a good point of reference to look at for the patch you are trying to develop here.

Comment 5

[Bill Gianopoulos [:WG9s]]

Oh and i do have Linux and windows builds including Fred's patch available at http://www.wg9s.com/mozilla/firefox/

Comment 6

[Marco Bonardo [:mak]]

oh cool, I'm going to dupe this there, and will provide the tests!

Comment 7

[Marco Bonardo [:mak]]

fwiw, my patch was pretty much similar (Apart that I used isDefaultNamespace), so no reason to conflict :)