Skip to content
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

Adds notes about controls and playback. #79

Merged
merged 5 commits into from
May 6, 2017

Conversation

markafoltz
Copy link
Contributor

Addresses Issue #48: Implementation guidance for browsers when a media element with controls is remoted.

  • Adds a NOTE that the remote playback state should be reflected in default media controls.
  • Adds a NOTE that local audio and video output for the media element should be paused.

The issue didn't discuss the disposition of text tracks (embedded in the media stream, or in <track> elements) so I crafted the note to refer to specifically audio and video.

index.html Outdated
<div class="note">
If the user agent is <a>exposing a user interface to the user</a>
for the media element (i.e., using default controls), the user agent
SHOULD convey the fact that remote playback is connected through an
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I don't think "remote playback is connected" is a valid expression, at least in the spec. Maybe replace with "remote playback state is connected" with links for both state and connected?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@avayvod
Copy link
Contributor

avayvod commented May 6, 2017

Thanks!

@avayvod avayvod merged commit 9ad2980 into w3c:gh-pages May 6, 2017
@markafoltz markafoltz deleted the issue-48-controls branch June 14, 2017 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants