Closed
Bug 874003
Opened 12 years ago
Closed 12 years ago
Remove preventBubble and preventCapture
Categories
(Core :: DOM: Events, defect)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla24
People
(Reporter: bruant.d, Assigned: emk)
Details
(Keywords: addon-compat, dev-doc-complete, site-compat)
Attachments
(1 file)
14.30 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
There is no plan in make a standard out of these Mozilla-specific methods. stopPropagation is what is used by developers (standard and better cross-browser compat) They have no reason to exist as properties on Event.prototype anymore.
Updated•12 years ago
|
Comment 1•12 years ago
|
||
Need to be careful especially with preventBubble. Traditionally it has been used oddly often.
Assignee | ||
Comment 2•12 years ago
|
||
Let's try and see what's going on early in the cycle. Currently preventBubble is no-op, so any code depending on preventBubble is broken anyway. Try result: https://tbpl.mozilla.org/?tree=Try&rev=0fe79854849d
Updated•12 years ago
|
Attachment #751836 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/343242181775
Flags: in-testsuite-
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/343242181775
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
Comment 5•12 years ago
|
||
Updated: https://developer.mozilla.org/en-US/docs/Web/API/Event https://developer.mozilla.org/en-US/docs/XPCOM_Interface_Reference/nsIDOMEvent https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_24 We should also consider removing (Bug 691151) or deprecating (Bug 726933) getPreventDefault. The nsIDOMEvent document says it has been deprecated since Gecko 16.0 but the bugs are still open.
Keywords: dev-doc-needed → dev-doc-complete
Assignee | ||
Comment 6•12 years ago
|
||
(In reply to Kohei Yoshino from comment #5) > https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_24 I think preventBubble and preventCapture should be explained in a separate clause. These methods are not derived from Netscape but from the earlier W3C draft. Chromium have never supported these methods from the start. And the replacement method is stopPropagation.
Comment 7•12 years ago
|
||
(In reply to Masatoshi Kimura [:emk] from comment #6) > I think preventBubble and preventCapture should be explained in a separate > clause. > These methods are not derived from Netscape but from the earlier W3C draft. > Chromium have never supported these methods from the start. And the > replacement method is stopPropagation. Thank you for the feedback. I just corrected the document.
You need to log in
before you can comment on or make changes to this bug.
Description
•