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

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

Issue 1458733003: Reland MD Audio Player. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: forcePlay -> forceToPlay 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 216bb44802d251bb32ca67aba672303abb9acdeb..e178163b812f6859225766681ab0d89e49e4d8fb 100644
--- a/ui/file_manager/audio_player/js/audio_player.js
+++ b/ui/file_manager/audio_player/js/audio_player.js
@@ -64,6 +64,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',
@@ -325,21 +338,21 @@ AudioPlayer.prototype.onKeyDown_ = function(event) {
* @type {number}
* @const
*/
-AudioPlayer.HEADER_HEIGHT = 33; // 32px + border 1px
+AudioPlayer.HEADER_HEIGHT = 36; // 32px + border 4px
/**
* Track height in pixels.
* @type {number}
* @const
*/
-AudioPlayer.TRACK_HEIGHT = 44;
+AudioPlayer.TRACK_HEIGHT = 48;
/**
* Controls bar height in pixels.
* @type {number}
* @const
*/
-AudioPlayer.CONTROLS_HEIGHT = 73; // 72px + border 1px
+AudioPlayer.CONTROLS_HEIGHT = 96;
/**
* Default number of items in the expanded mode.
« no previous file with comments | « ui/file_manager/audio_player/icons/audio-player-favicon-32.png ('k') | ui/file_manager/audio_player/js/background.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698