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

Doc fix #4790

Merged
merged 2 commits into from
Dec 4, 2017
Merged

Doc fix #4790

merged 2 commits into from
Dec 4, 2017

Conversation

thijstriemstra
Copy link
Contributor

Text tracks are not video-only and can be used for audio as well.

Text tracks are not video-only and can be used for audio as well.
@@ -1,6 +1,6 @@
# Text Tracks

Text tracks are a feature of HTML5 video for displaying time-triggered text to the viewer. Video.js offers a cross-browser implementation of text tracks.
Text tracks are a feature of HTML5 for displaying time-triggered text to the user. Video.js offers a cross-browser implementation of text tracks.
Copy link
Member

Choose a reason for hiding this comment

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

makes to remove video there, viewer does imply video but I'm not sure how well user works. Maybe end-user? Since user often implies the user of Video.js, aka, the developer.

@gkatsev gkatsev added the patch This PR can be added to a patch release. label Dec 4, 2017
@gkatsev gkatsev merged commit 1672a6d into videojs:master Dec 4, 2017
@gkatsev
Copy link
Member

gkatsev commented Dec 4, 2017

Thanks @thijstriemstra

@thijstriemstra thijstriemstra deleted the patch-2 branch December 4, 2017 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch This PR can be added to a patch release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants