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

Unified Diff: chrome/browser/resources/file_manager/background/js/background.js

Issue 185653014: [AudioPlayer] Fix a bug on changing 'expanded' status (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Adressed the comment Created 6 years, 9 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
« no previous file with comments | « chrome/browser/resources/file_manager/audio_player/js/audio_player.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/file_manager/background/js/background.js
diff --git a/chrome/browser/resources/file_manager/background/js/background.js b/chrome/browser/resources/file_manager/background/js/background.js
index b17c5a78ace59ac733fb8daa57bdecdf3f7e0b3b..8c0a3874781ef1e496d273c950c02885c8187cb1 100644
--- a/chrome/browser/resources/file_manager/background/js/background.js
+++ b/chrome/browser/resources/file_manager/background/js/background.js
@@ -764,9 +764,12 @@ audioPlayerInitializationQueue.run(function(callback) {
var audioPlayerCreateOptions = Object.freeze({
type: 'panel',
hidden: true,
- minHeight: newAudioPlayerEnabled ? 116 : (35 + 58),
+ minHeight:
+ newAudioPlayerEnabled ?
+ (44 + 73) : // 44px: track, 73px: controller
+ (35 + 58), // 35px: track, 58px: controller
minWidth: newAudioPlayerEnabled ? 292 : 280,
- height: newAudioPlayerEnabled ? 116 : (35 + 58), // default collapsed
+ height: newAudioPlayerEnabled ? (44 + 73) : (35 + 58), // collapsed
width: newAudioPlayerEnabled ? 292 : 280,
});
« no previous file with comments | « chrome/browser/resources/file_manager/audio_player/js/audio_player.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698