Simplify text regarding event handlers #309

Closed
ibc opened this Issue Dec 25, 2015 · 2 comments

Projects

None yet

2 participants

@ibc
Contributor
ibc commented Dec 25, 2015

This event handler, of event handler type xxxxxx, uses the RTCXxxXxxxXxxxxEvent interface. It must be supported by all objects implementing the RTCXxxXxxxx interface.

Do we really need such an exact text in the definition of all the event handlers within the spec? Yes, WebRTC 1.0 does it, but it becomes really annoying/boring for the reader (IMHO).

@aboba aboba added 1.0 1.1 labels Dec 26, 2015
@aboba
Contributor
aboba commented Jan 12, 2016

WebRTC 1.0 has changed much of the text relating to event handlers, so that a lot of the above text is now gone. The ORTC specification should do the same.

@aboba aboba removed the to-do-next-draft label Feb 21, 2016
@aboba
Contributor
aboba commented Jul 7, 2016

Went through all the event handler text. With the adoption of the WebRTC 1.0 "statechange" event, quite a bit of the event handler text could be deleted. Also, I do not think it makes sense for an EventHandler to be nullable. The event table also needed to be updated to reflect the "statechange" event.

@aboba aboba added the PR exists label Jul 7, 2016
@aboba aboba closed this Jul 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment