Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(606)

Unified Diff: ui/file_manager/audio_player/js/audio_player.js

Issue 1415953006: Fix accessibility issues in AudioPlayer and VideoPlayer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix query for repeat button. Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/file_manager/audio_player/js/audio_player.js
diff --git a/ui/file_manager/audio_player/js/audio_player.js b/ui/file_manager/audio_player/js/audio_player.js
index ea5fd263e79f3d1029b001bd4c59e7779f09872e..fb09c33e3b2b86c8306634fa0f19c36327829fe8 100644
--- a/ui/file_manager/audio_player/js/audio_player.js
+++ b/ui/file_manager/audio_player/js/audio_player.js
@@ -68,6 +68,19 @@ function AudioPlayer(container) {
this.offlineString_ = strings['AUDIO_OFFLINE'];
AudioPlayer.TrackInfo.DEFAULT_ARTIST =
strings['AUDIO_PLAYER_DEFAULT_ARTIST'];
+ // Pass translated labels to the AudioPlayerElement.
+ this.player_.ariaLabels = {
+ volumeSlider: strings['AUDIO_PLAYER_OPEN_VOLUME_SLIDER_BUTTON_LABEL'],
+ shuffle: strings['AUDIO_PLAYER_SHUFFLE_BUTTON_LABEL'],
+ repeat: strings['AUDIO_PLAYER_REPEAT_BUTTON_LABEL'],
+ previous: strings['MEDIA_PLAYER_PREVIOUS_BUTTON_LABEL'],
+ play: strings['MEDIA_PLAYER_PLAY_BUTTON_LABEL'],
+ pause: strings['MEDIA_PLAYER_PAUSE_BUTTON_LABEL'],
+ next: strings['MEDIA_PLAYER_NEXT_BUTTON_LABEL'],
+ volume: strings['AUDIO_PLAYER_OPEN_VOLUME_SLIDER_BUTTON_LABEL'],
+ playList: strings['AUDIO_PLAYER_OPEN_PLAY_LIST_BUTTON_LABEL'],
+ seekSlider: strings['MEDIA_PLAYER_SEEK_SLIDER_LABEL']
+ };
}.bind(this));
this.volumeManager_.addEventListener('externally-unmounted',

Powered by Google App Engine
This is Rietveld 408576698