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

Unified Diff: chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html

Issue 2133793003: Unify iron-dropdown CSS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More feedback Created 4 years, 5 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
« no previous file with comments | « no previous file | chrome/browser/resources/settings/languages_page/languages_page.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html
diff --git a/chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html b/chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html
index 500c5f9b411764152e7b1d9d9471a0b65a348a96..d2d9c489676cfaf23839cf5184154a95c7b417e3 100644
--- a/chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html
+++ b/chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html
@@ -5,9 +5,10 @@
<link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.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-item/paper-item.html">
+<link rel="import" href="/settings_shared_css.html">
<dom-module id="bluetooth-device-list-item">
- <style>
+ <style include="settings-shared">
iron-icon {
-webkit-padding-start: 10px;
}
@@ -19,12 +20,6 @@
span.name[connected] {
font-weight: 500;
}
-
- /* TODO(dbeam): combine with other similar styles. */
- .dropdown-content {
- background: white;
- box-shadow: 0 2px 6px grey;
- }
</style>
<template>
<div class="flex layout horizontal center" dropdown$="[[dropdownOpened]]"
@@ -36,16 +31,13 @@
</iron-icon>
<span class="flex"></span>
<span hidden$="[[!device.connecting]]">$i18n{bluetoothConnecting}</span>
- <!-- This <div on-tap> is not [actionable] because its effect is not user
- visible. -->
+ <!-- <div on-tap> is usually [actionable], but this on-tap is just to
+ prevent triggering |itemTapped_| in the containing div. -->
<div hidden$="[[!device.paired]]" on-tap="doNothing_">
<paper-icon-button icon="cr:more-vert" toggles
active="{{dropdownOpened}}">
</paper-icon-button>
- <!-- TODO(dbeam): does this need [actionable] if all the controls are
- paper-items. Conversely, should the on-tap be at this level? -->
- <iron-dropdown opened="{{dropdownOpened}}" on-tap="menuSelected_"
- actionable>
+ <iron-dropdown opened="{{dropdownOpened}}" on-tap="menuSelected_">
<div class="dropdown-content">
<paper-item id="connect" hidden$="[[device.connected]]">
$i18n{bluetoothConnect}
« no previous file with comments | « no previous file | chrome/browser/resources/settings/languages_page/languages_page.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698