-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Highlighting errorCode 701 #1458
Conversation
Fix for Issue #1281
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure this addresses Ekr's concern; we may want to just wait until we hear a suggestion from him.
@@ -4409,6 +4409,13 @@ | |||
<p>The <code>icecandidateerror</code> event of the RTCPeerConnection | |||
uses the <code><a>RTCPeerConnectionIceErrorEvent</a></code> | |||
interface.</p> | |||
<p>An <code><a>RTCPeerConnectionIceErrorEvent</a></code> event MUST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two "MUST"s in a row here.
@@ -4488,11 +4495,6 @@ | |||
<p>The <code>errorCode</code> attribute is the numeric STUN | |||
error code returned by the STUN or TURN server | |||
[[STUN-PARAMETERS]].</p> | |||
<p>If no host candidate can reach the server, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this text should be removed. It's normal to expect an attribute's description to describe all its possible values, without having to look elsewhere (in this case, "elsewhere" is the description of the containing dictionary).
Closing the issue as well as this PR since moving the text seems to create its own set of problems. |
Waiting for response from EKR (do not merge).
Fix for Issue #1281
Preview | Diff