Status
()
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Tracking
Firefox Tracking Flags
(firefox53 fixed)
Details
Attachments
(1 attachment)
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•2 years ago
|
||
Created attachment 8826043 [details] [diff] [review] Introduce DocumentAndElementEventHandlers to more closely align with the HTML spec
Attachment #8826043 -
Flags: review?(bkelly)
Comment 2•2 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•2 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•2 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•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/426c5f5536bf
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in
before you can comment on or make changes to this bug.
Description
•