-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add missing definitions, fix ReSpec & validator warnings #72
Conversation
Fixes the following ReSpec warnings: No <dfn> for RemotePlayback. No <dfn> for RemotePlaybackState. No <dfn> for RemotePlaybackAvailabilityCallback. pre elements don't need a 'highlight' class anymore.
<a for> -> <a data-link-for> <dfn for> -> <dfn data-dfn-for> See: https://github.com/w3c/respec/wiki/data-link-for https://github.com/w3c/respec/wiki/data-dfn-for
index.html
Outdated
@@ -440,129 +498,184 @@ | |||
|
|||
callback RemotePlaybackAvailabilityCallback = void(boolean available); | |||
</pre> | |||
<p> | |||
A <dfn>RemotePlayback</dfn> object represents a <a>remote playback | |||
device</a>. |
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 about that but I will let @avayvod decide :)
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.
Right, I will defer to the editors on the proper definition of this thing :-)
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'd rather say something like: A RemotePlayback object is an interface allowing the page to detect, establish connection to and control playback on remote playback devices.
@avayvod any concerns in merging this PR? |
Modulo the definition of RemotePlayback object, lgtm. |
Please push your fix to this branch and merge. |
Thanks for taking care of this, Anssi! @anssiko |
PTAL @mounirlamouri @avayvod
Btw. The spec is on TAG's agenda for tomorrow -- this PR adds some polish to it :-)