Closed
Bug 1330457
Opened 6 years ago
Closed 6 years ago
Introduce DocumentAndElementEventHandlers
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
Attachments
(1 file)
5.32 KB,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
See https://html.spec.whatwg.org/multipage/webappapis.html#documentandelementeventhandlers This doesn't really change web-visible behavior; just aligns our IDL more closely with the spec.
![]() |
Assignee | |
Comment 1•6 years ago
|
||
Attachment #8826043 -
Flags: review?(bkelly)
Comment 2•6 years ago
|
||
Comment on attachment 8826043 [details] [diff] [review] Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec Review of attachment 8826043 [details] [diff] [review]: ----------------------------------------------------------------- Can you point out where the SVGElement spec for these event handlers is? ::: dom/webidl/SVGElement.webidl @@ +28,5 @@ > [Throws] void blur(); > }; > > SVGElement implements GlobalEventHandlers; > +SVGElement implements DocumentAndElementEventHandlers; I don't see this in the spec. Maybe I have the wrong link. Looking at: https://svgwg.org/svg2-draft/types.html#InterfaceSVGElement
Attachment #8826043 -
Flags: review?(bkelly)
![]() |
Assignee | |
Comment 3•6 years ago
|
||
Comment on attachment 8826043 [details] [diff] [review] Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec The SVG spec situation here is a mess. See https://www.w3.org/Graphics/SVG/WG/track/actions/3283 which hasn't happened yet (and is linked from <https://svgwg.org/svg2-draft/interact.html#SVGEvents>), but in general their intent is that the set of event handlers on HTML and SVG elements be identical.
Attachment #8826043 -
Flags: review?(bkelly)
Comment 4•6 years ago
|
||
Comment on attachment 8826043 [details] [diff] [review] Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec Review of attachment 8826043 [details] [diff] [review]: ----------------------------------------------------------------- Thanks.
Attachment #8826043 -
Flags: review?(bkelly) → review+
Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/426c5f5536bf Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec. r=bkelly
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/426c5f5536bf
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•4 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•