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

Unified Diff: ui/file_manager/audio_player/elements/control_panel.html

Issue 1491543002: AudioPlayer: Update layout to show the volume slider without using popup. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/elements/control_panel.html
diff --git a/ui/file_manager/audio_player/elements/control_panel.html b/ui/file_manager/audio_player/elements/control_panel.html
index 6e9df2619e71ddf6fb9b03368a3878aa272e1f86..4520b5778eefe8b4dc52ab8351fb8baabad0cb09 100644
--- a/ui/file_manager/audio_player/elements/control_panel.html
+++ b/ui/file_manager/audio_player/elements/control_panel.html
@@ -50,24 +50,6 @@
on-click="nextClick">
</files-icon-button>
- <div id="volumeContainer"
- class="default-hidden"
- anchor-point="bottom center">
- <volume-controller id="volumeSlider" value="{{volume}}"
yawano 2015/12/01 07:51:31 If volume-controller is no longer used, please del
fukino 2015/12/02 04:52:02 Done.
- width="32" height="85" value="50">
- </volume-controller>
-
- <polymer-anchor-point id="anchorHelper"></polymer-anchor-point>
- </div>
-
- <!-- Volume button in the bottom line. -->
- <files-icon-button toggles
- id="volumeButton"
- class="volume media-button toggle"
- anchor-point="bottom center"
- active="{{volumeSliderShown}}">
- </files-icon-button>
-
<!-- Playlist button in the bottom line. -->
<files-icon-button toggles
id="playList"
@@ -75,12 +57,21 @@
active="{{expanded}}">
</files-icon-button>
</div>
- <div class="lower-controls time-controls">
+ <div class="lower-controls time-volume-controls">
+ <!-- Play/pause button and seek slider in the bottom line. -->
<div class="time-container">
<div class="time-spacer">[[computeTimeString_(duration, duration)]]</div>
<div class="time">[[computeTimeString_(time, duration)]]</div>
</div>
<paper-slider id="timeSlider" max="[[duration]]" value="{{time::change}}"></paper-slider>
+
+ <!-- Volume button and slider in the bottom line. -->
+ <files-icon-button
+ id="volumeButton"
+ class="volume media-button"
+ on-click="volumeClick">
+ </files-icon-button>
+ <paper-slider id="volumeSlider"></paper-slider>
</div>
</div>
</template>

Powered by Google App Engine
This is Rietveld 408576698