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

Unified Diff: chrome/browser/resources/chromeos/emulator/battery_settings.html

Issue 2938933002: WebUI: swap paper-dropdown-menu out for md-select. (Closed)
Patch Set: switch md_user_manager/create_profile dropdown Created 3 years, 6 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/chromeos/emulator/battery_settings.html
diff --git a/chrome/browser/resources/chromeos/emulator/battery_settings.html b/chrome/browser/resources/chromeos/emulator/battery_settings.html
index 9eceac96f0bddbc316c7a467f1608215a9ad8e9b..583d0d30fef400c7bc9714a27adb621b0a56e06d 100644
--- a/chrome/browser/resources/chromeos/emulator/battery_settings.html
+++ b/chrome/browser/resources/chromeos/emulator/battery_settings.html
@@ -1,4 +1,5 @@
<link rel="import" href="chrome://resources/html/polymer.html">
+<link rel="import" href="chrome://resources/html/md_select_css.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-collapse/iron-collapse.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout-classes.html">
<link rel="import" href="chrome://resources/polymer/v1_0/neon-animation/neon-animatable-behavior.html">
@@ -17,7 +18,8 @@
<template>
<!-- TODO(michaelpg): Wrap the line below to fit within the 80-char limit.
See https://github.com/Polymer/polymer/pull/3668. -->
- <style include="device-emulator-shared-styles iron-flex iron-flex-alignment iron-positioning">
+ <style include="device-emulator-shared-styles iron-flex iron-flex-alignment
+ iron-positioning md-select">
</style>
<div class="layout vertical">
<div class="element-label">
@@ -74,14 +76,19 @@
<div hidden$="[[isDualRole(item)]]">Source</div>
</td>
<td class="control-cell">
- <paper-dropdown-menu class="device-class-group"
- disabled="[[!item.variablePower]]">
- <paper-listbox selected="{{item.power}}"
- class="dropdown-content" attr-for-selected="data-value">
- <paper-item data-value="high">High</paper-item>
- <paper-item data-value="low">Low</paper-item>
- </paper-listbox>
- </paper-dropdown-menu>
+ <div class="md-select-wrapper">
+ <select class="md-select" disabled$="[[!item.variablePower]]"
+ on-change="onPowerChanged">
+ <template is="dom-repeat" items="[[powerOptions]]"
dpapad 2017/06/15 01:35:36 Is it necessary to use a dom-repeat for static con
scottchen 2017/06/15 23:50:51 Discussed offline with dpapad, hard-coding this in
+ as="option">
+ <option value="[[option]]"
+ selected="[[isPowerEqual(item.power, option)]]">
+ [[option]]
+ </option>
+ </template>
+ </select>
+ <span class="md-select-underline"></span>
+ </div>
</td>
</template>
</tbody>

Powered by Google App Engine
This is Rietveld 408576698