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

Unified Diff: chrome/browser/resources/file_manager/audio_player/elements/track_list.css

Issue 144883002: [Files.app] Initial implementation of new audio player (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Separete the image files from this patch Created 6 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: chrome/browser/resources/file_manager/audio_player/elements/track_list.css
diff --git a/chrome/browser/resources/file_manager/audio_player/elements/track_list.css b/chrome/browser/resources/file_manager/audio_player/elements/track_list.css
new file mode 100644
index 0000000000000000000000000000000000000000..09c4fac8454c6942dd2a62051bc24e672d61bb85
--- /dev/null
+++ b/chrome/browser/resources/file_manager/audio_player/elements/track_list.css
@@ -0,0 +1,131 @@
+/* Copyright (c) 2012 The Chromium Authors. All rights reserved.
mtomasz 2014/01/23 08:05:09 nit: 2014
yoshiki 2014/01/24 15:34:19 Done.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file. */
+
mtomasz 2014/01/23 08:05:09 The last track is not fully visible, if there is a
yoshiki 2014/01/24 15:34:19 Fixed the height of bottom margin.
+:host {
+ -webkit-box-align: center;
+ -webkit-box-orient: vertical;
+ -webkit-box-pack: start;
+ background: #f5f5f5;
+ bottom: 62px; /* Room for the controls bar. */
+ color: #3d3d3d;
+ cursor: default;
+ display: -webkit-box;
+ font-family: Open Sans, Droid Sans Fallback, sans-serif;
+ font-size: 10pt;
+ left: 0;
+ overflow-x: hidden;
+ overflow-y: auto;
+ position: absolute;
+ right: 0;
+ top: 0;
+}
+
+/* Common properties for track containers. */
+#track-list,
mtomasz 2014/01/23 08:05:09 Since empty can we remove?
yoshiki 2014/01/24 15:34:19 Done.
+#track-stack {
+}
+
+/* Scrollable list of tracks.
+ * Displayed in expanded mode if there is more than one track. */
+#track-list {
+
+}
+
+/* A single track container.
+ * Displayed in the compact mode or when there is only one track. */
+.track-stack {
+ height: 58px;
+}
+
+.audio-player.collapsed > #track-list,
+.audio-player.single_track > #track-list,
+.audio-player:not(.collapsed):not(.single-track) > .track-stack {
+ opacity: 0;
+ pointer-events: none;
+}
+
+/* Track item. */
+.track {
+ -webkit-box-align: center;
+ -webkit-box-orient: horizontal;
+ -webkit-box-pack: start;
+ display: -webkit-box;
+ height: 44px;
+ padding-left: 20px;
mtomasz 2014/01/23 08:05:09 nit: The padding is too large comparing to specs.
yoshiki 2014/01/24 15:34:19 Removed the padding in .track.data. Done.
+ width: 100%;
+}
+
+track-list:not([expanded]) > .track:not([active='true']) {
+ display: none;
+}
+
+/* In the expanded mode the selected track is highlighted. */
+.track[active='true'] {
+ background-color: rgb(66, 129, 244);
+}
+
+/* In the compact mode all tracks are in the same position, only the selected
+ is visible.*/
+.track-stack > .track {
+ position: absolute;
+ top: 0;
+}
+
+.track-stack > .track.selected {
+ z-index: 1;
mtomasz 2014/01/23 08:05:09 Why z-index is needed?
yoshiki 2014/01/24 15:34:19 Sorry, removed unnecessary styles.
+}
+
+/* Opacity transition is controlled differently for the text and the artwork.
+ * Text transition looks better if fade-in and fade-out go in parallel.
+ * For the artwork we start fading out the old icon only after the new one
+ * is completely opaque (otherwise when transitioning between identical icons
+ * we see some fading because the background transpires). */
+.track-stack > .track:not(.selected) .data,
+.track-stack > .track:not(.visible) .art {
+ opacity: 0;
+ transition: opacity 220ms ease-out;
+}
+
+/* Track data. */
+
+.track .data {
+ -webkit-box-flex: 1;
+ -webkit-box-orient: vertical;
+ -webkit-box-pack: center;
+ display: -webkit-box;
+ margin-left: 8px;
+ margin-right: 4px;
+}
+
+.track .data .data-title,
+.track .data .data-artist {
+ font-size: 12px;
+ overflow: hidden;
+ text-overflow: ellipsis;
+ white-space: nowrap;
+}
+
+.track .data .data-title {
+ color: #343434;
+ font-weight: bold;
+}
+
+.track .data .data-artist {
+ color: #969696;
+}
+
+.track[active='true'] .data .data-title,
+.track[active='true'] .data .data-artist {
+ color: #fff;
+}
+
+/* TODO(kaznacheev): Set to 20px when the audio player is title-less. */
+.single-track .data-title {
+ padding-right: 0;
+}
+
+/* TODO(kaznacheev): Set to 50px when the audio player is title-less. */
+.collapsed:not(.single-track) .data-title {
+ padding-right: 20px;
+}

Powered by Google App Engine
This is Rietveld 408576698