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

Side by Side Diff: chrome/browser/resources/file_manager/audio_player/elements/volume_controller.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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 /* Cpyright (c) 2012 The Chromium Authors. All rights reserved.
2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file. */
4
mtomasz 2014/01/23 08:05:09 nit: Shadow for the volume controller is too stron
yoshiki 2014/01/24 15:34:19 Done.
5 :host {
6 background: white;
7 display: block;
8 height: 100px;
9 position: relative;
10 width: 32px;
11 }
12
13 #background {
14 height: 100%; /* will be overridden by javascript */
15 left: 0;
16 position: absolute;
17 top: 0;
18 width: 100%;
19 }
20
21 input[type='range'] {
22 -webkit-appearance: none !important;
23 -webkit-transform: rotate(90deg);
24 background: transparent;
25 outline: none;
26 position: absolute;
27 z-index: 2;
28 }
29
30 input[type='range']::-webkit-slider-thumb {
31 -webkit-appearance: none;
32 -webkit-transform: rotate(-90deg);
33 background-image: -webkit-image-set(
34 url(../assets/100/player_timeline_handler.png) 1x,
35 url(../assets/100/player_timeline_handler.png) 2x);
mtomasz 2014/01/23 08:05:09 100 -> 200, please double check other places too.
yoshiki 2014/01/24 15:34:19 Done.
36 background-position: 50% 50%;
37 background-repeat: no-repeat no-repeat;
38 height: 24px;
39 position: relative;
40 width: 24px;
41 z-index: 2;
42 }
43
44 #bar {
45 background: #000;
46 bottom: 14px;
47 position: absolute;
48 top: 14px;
49 z-index: 1;
mtomasz 2014/01/23 08:05:09 Is the z-index needed?
yoshiki 2014/01/24 15:34:19 Removed.
50 }
51
52 #bar .filled {
53 background: #aaa;
54 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698