Closed Bug 706406 Opened 11 years ago Closed 11 years ago

<maction> doesn't work when click event is called stopPropagation()

Categories

(Core :: MathML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: masayuki, Assigned: masayuki)

References

()

Details

Attachments

(1 file)

Load the URI, you cannot toggle the maction's content.
Attached patch PatchSplinter Review
This also fixes another bug the frame doesn't check defaultPrevented value.
Attachment #577930 - Flags: review?(karlt)
Attachment #577930 - Flags: review?(bugs)
Attachment #577930 - Flags: review?(bugs) → review+
(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #1)
> Created attachment 577930 [details] [diff] [review] [diff] [details] [review]
> Patch
> 
> This also fixes another bug the frame doesn't check defaultPrevented value.

Does this fix bug 433774?
Attachment #577930 - Flags: review?(karlt) → review+
(In reply to Frédéric Wang from comment #2)
> (In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #1)
> > Created attachment 577930 [details] [diff] [review] [diff] [details] [review] [diff] [details] [review]
> > Patch
> > 
> > This also fixes another bug the frame doesn't check defaultPrevented value.
> 
> Does this fix bug 433774?

On the testcase, no. The click event is fired on <mn> rather than <maction>. So, the testcase doesn't make sense.
But that is a good point, I'll land the patch without defaultPrevented code, and I'll attach a new patch to the bug.
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8192280c3b3
Flags: in-testsuite+
Whiteboard: [inbound]
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/a8192280c3b3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Blocks: maction
You need to log in before you can comment on or make changes to this bug.