Skip to content

Add MediaTranscripts to MediaMetadata (#370)#371

Open
yrw-google wants to merge 2 commits intow3c:mainfrom
yrw-google:main
Open

Add MediaTranscripts to MediaMetadata (#370)#371
yrw-google wants to merge 2 commits intow3c:mainfrom
yrw-google:main

Conversation

@yrw-google
Copy link
Copy Markdown

@yrw-google yrw-google commented Mar 25, 2026

This adds the MediaTranscripts and MediaTranscript interfaces in order to add a list of MediaTranscripts to MediaMetadata, which allows sites to provide high-quality transcripts as described in #370.


Preview | Diff

Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Copy link
Copy Markdown
Member

@domfarolino domfarolino left a comment

Choose a reason for hiding this comment

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

Please find other instances of "should" that are in descriptive documentation of things enforced by Web IDL, and incorporate them.

Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Copy link
Copy Markdown
Member

@domfarolino domfarolino left a comment

Choose a reason for hiding this comment

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

I think the last thing that stands out to me is instances of "MUST" and "SHOULD" should be "must" and "should", without the capitalization, per ordinary WHATWG/W3C spec conventions.

Comment thread index.bs Outdated
@yrw-google
Copy link
Copy Markdown
Author

I think the last thing that stands out to me is instances of "MUST" and "SHOULD" should be "must" and "should", without the capitalization, per ordinary WHATWG/W3C spec conventions.

I've updated the "must" added in this PR to be uncapitalized, but the other instances are out of scope for this PR.

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