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

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

Issue 1657623002: Implement "expand/collapse artwork" button in audio player. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: sorting css prop. Created 4 years, 11 months 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/elements/audio_player.js
diff --git a/ui/file_manager/audio_player/elements/audio_player.js b/ui/file_manager/audio_player/elements/audio_player.js
index 6800dd0562d6766355d45517c826eded3fb02f3b..53da08f4eefe841dbd8144e7184bcfe8a1edeb7a 100644
--- a/ui/file_manager/audio_player/elements/audio_player.js
+++ b/ui/file_manager/audio_player/elements/audio_player.js
@@ -137,6 +137,7 @@ Polymer({
var currentTrack = this.$.trackList.getCurrentTrack();
if(currentTrack && currentTrack != this.$.trackInfo.track){
this.$.trackInfo.track = currentTrack;
+ this.$.trackInfo.artworkAvailable = !!currentTrack.artworkUrl;
}
if (currentTrack && currentTrack.url != this.$.audio.src) {
this.$.audio.src = currentTrack.url;
@@ -377,11 +378,14 @@ Polymer({
this.$.trackInfo.notifyPath('track.title', this.tracks[index].title);
this.$.trackInfo.notifyPath('track.artist', this.tracks[index].artist);
var artworkUrl = this.tracks[index].artworkUrl;
- if (!artworkUrl || artworkUrl !== '')
+ if (!artworkUrl && artworkUrl !== '') {
this.$.trackInfo.notifyPath('track.artworkUrl',
this.tracks[index].artworkUrl);
- else
+ this.$.trackInfo.artworkAvailable = true;
+ } else {
this.$.trackInfo.notifyPath('track.artworkUrl', undefined);
+ this.$.trackInfo.artworkAvailable = false;
+ }
}
},
« no previous file with comments | « ui/file_manager/audio_player/css/audio_player.css ('k') | ui/file_manager/audio_player/elements/track_info_panel.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698