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

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

Issue 2938933002: WebUI: swap paper-dropdown-menu out for md-select. (Closed)
Patch Set: appease the compiler god 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..75311e900b47f2c227d54fa30b9fe39e41278126 100644
--- a/chrome/browser/resources/chromeos/emulator/battery_settings.html
+++ b/chrome/browser/resources/chromeos/emulator/battery_settings.html
@@ -1,13 +1,11 @@
<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">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-checkbox/paper-checkbox.html">
-<link rel="import" href="chrome://resources/polymer/v1_0/paper-dropdown-menu/paper-dropdown-menu.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-input/paper-input.html">
-<link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.html">
-<link rel="import" href="chrome://resources/polymer/v1_0/paper-listbox/paper-listbox.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-radio-button/paper-radio-button.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-radio-group/paper-radio-group.html">
<link rel="import" href="icons.html">
@@ -17,7 +15,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 +73,18 @@
<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" value="[[item.power]]">
+ <option value="high">
dpapad 2017/06/16 00:55:19 Nit: Can we fit those into one line? <option valu
scottchen 2017/06/16 01:53:20 Done.
+ High
+ </option>
+ <option value="low">
+ Low
+ </option>
+ </select>
+ <span class="md-select-underline"></span>
+ </div>
</td>
</template>
</tbody>

Powered by Google App Engine
This is Rietveld 408576698