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

Convert HTMLMediaElements links to autolinks #269

Merged
merged 1 commit into from
Mar 15, 2021
Merged

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Mar 9, 2021

This update converts all explicit references to the HTML spec to Bikeshed's autolinks syntax. To render the name of the interface along with the name of the attribute or method, the "visible text" syntax is used (the text after |).

The notions of "media error code", official playback position", "poster frame" and "timeline offset" are not HTMLMediaElement attributes but rather internal HTML concepts whose definitions are not currently exported. Longer term, we should make sure that the HTML spec exports these terms (unless we don't need them anymore).

The text incorrect referred to a stalled attribute that does not exist, instead of to the stalled event. Text updated accordingly.

This would address #264.


Preview | Diff

This update converts all explicit references to the HTML spec to Bikeshed's
autolinks syntax. To render the name of the interface along with the name of
the attribute or method, the "visible text" syntax is used (the text after `|`).

The notions of "media error code", official playback position", "poster frame"
and "timeline offset" are not HTMLMediaElement attributes but rather internal
HTML concepts whose definitions are not currently exported. Longer term, we
should make sure that the HTML spec exports these terms (unless we don't need
them anymore).

The text incorrect referred to a `stalled` attribute that does not exist,
instead of to the `stalled` event. Text updated accordingly.
@tidoust tidoust requested a review from markafoltz March 9, 2021 11:58
Copy link
Contributor

@markafoltz markafoltz left a comment

Choose a reason for hiding this comment

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

Brilliant, thank you @tidoust!

@louaybassbouss
Copy link
Contributor

I am going to merge this PR since it is approved by @mfoltzgoogle

@louaybassbouss louaybassbouss merged commit a332409 into master Mar 15, 2021
github-actions bot added a commit that referenced this pull request Mar 15, 2021
…LMediaElements links to autolinks

SHA: a332409
Reason: push, by @louaybassbouss

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@markafoltz markafoltz deleted the autolinks branch October 25, 2021 18:57
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.

3 participants