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

Changed Audio Track button title to 'Audio Track' and modified en.json #3574

Conversation

jbarabander
Copy link
Contributor

@jbarabander jbarabander commented Aug 24, 2016

Description

Per discussion on #3565 I changed the audio track title to be 'Audio Track' instead of just 'Audio'
Additionally I updated the en.json.

Specific Changes proposed

Modified en.json to have an entry for 'Audio Track'.
Modified the AudioTrackButton components controlText_ to be 'Audio Track'

Requirements Checklist

  • Feature implemented / Bug fixed
  • Reviewed by Two Core Contributors

@jbarabander jbarabander changed the title Changed button title to Audio Track and modified en.json to reflect this Changed button title to Audio Track and modified en.json Aug 24, 2016
@jbarabander jbarabander changed the title Changed button title to Audio Track and modified en.json Changed Audio Track button title to 'Audio Track' and modified en.json Aug 24, 2016
@gkatsev
Copy link
Member

gkatsev commented Aug 25, 2016

LGTM

@gkatsev gkatsev added patch This PR can be added to a patch release. needs: LGTM Needs one or more additional approvals labels Aug 25, 2016
@brandonocasey
Copy link
Contributor

LGTM

@misteroneill misteroneill added confirmed and removed needs: LGTM Needs one or more additional approvals labels Aug 25, 2016
gkatsev pushed a commit to gkatsev/video.js that referenced this pull request Aug 25, 2016
@gkatsev
Copy link
Member

gkatsev commented Aug 25, 2016

Merged to stable. Thanks!

@gkatsev gkatsev closed this Aug 25, 2016
@jbarabander
Copy link
Contributor Author

jbarabander commented Aug 25, 2016

@gkatsev: Awesome!!! Thanks so much for all your help and encouragement!

@gkatsev
Copy link
Member

gkatsev commented Aug 25, 2016

@jbarabander happy to help. Thanks for your effort as well!

helenjer pushed a commit to helenjer/video.js that referenced this pull request Sep 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed patch This PR can be added to a patch release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants