'formdata' event should bubble. #4422
Conversation
This is a follow-up of whatwg#4239, and fixes an issue pointed out in web-platform-tests/wpt#14637
| @@ -56651,7 +56651,8 @@ fur | |||
| <li><p><span data-x="concept-event-fire">Fire an event</span> named | |||
| <code data-x="event-formdata">formdata</code> at <var>form</var> using | |||
| <code>FormDataEvent</code>, with the <code data-x="dom-FormDataEvent-formData">formData</code> | |||
| attribute initialized to <var>form data</var>.</p></li> | |||
| attribute initialized to <var>form data</var> and the | |||
| <code data-x="dom-Event-bubbles">bubbles</code> attribute initialized to true.</p></li> | |||
There was a problem hiding this comment.
I think we might want to set boolean bubbles = true; in the IDL init dictionary instead. In Gecko, events are automatically generated, so I can’t implement this in the way specified here.
There was a problem hiding this comment.
I don't think it's possible in WebIDL.
There was a problem hiding this comment.
Indeed, this is the standard pattern for firing events across the spec and web. You can't and shouldn't change the dictionary default, as that would affect user-created FormDataEvents.
I'm not sure how Gecko implements the hundreds of other cases in the spec which are worded this way, but it's worth looking in to.
There was a problem hiding this comment.
Yeah, you are right: can’t override the underlying EventInit default. That’s alright, I’ll figure it out on the Gecko side.
This is a follow-up of #4239, and
fixes an issue pointed out in web-platform-tests/wpt#14637
A test for the new behavior was already merged to WPT.
/form-control-infrastructure.html ( diff )