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

Side by Side Diff: chrome/browser/resources/settings/site_settings/media_picker.html

Issue 2128173002: Move settings_dropdown_menu styling to settings_shared_css. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 unified diff | Download patch
OLDNEW
1 <link rel="import" href="chrome://resources/html/polymer.html"> 1 <link rel="import" href="chrome://resources/html/polymer.html">
2 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dropdown-menu/pap er-dropdown-menu.html"> 2 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dropdown-menu/pap er-dropdown-menu.html">
3 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.h tml"> 3 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.h tml">
dpapad 2016/07/07 21:03:55 No longer needed.
stevenjb 2016/07/07 21:39:06 Done.
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-listbox/paper-lis tbox.html"> 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-listbox/paper-lis tbox.html">
5 <link rel="import" href="/settings_shared_css.html"> 5 <link rel="import" href="/settings_shared_css.html">
6 6
7 <dom-module id="media-picker"> 7 <dom-module id="media-picker">
8 <template> 8 <template>
9 <style include="settings-shared"> 9 <style include="settings-shared">
10 :host { 10 :host {
11 display: block; 11 display: block;
12 } 12 }
13
14 paper-dropdown-menu {
15 --paper-input-container-input: {
16 font-size: inherit;
17 };
18
19 --paper-font-caption: {
20 font-size: inherit;
21 }
22 }
23 </style> 13 </style>
24 <div class="settings-box" id="picker" hidden> 14 <div class="settings-box" id="picker" hidden>
25 <paper-dropdown-menu> 15 <paper-dropdown-menu>
26 <paper-listbox id="mediaPicker" class="dropdown-content" 16 <paper-listbox id="mediaPicker" class="dropdown-content"
27 attr-for-selected="media-picker-value" 17 attr-for-selected="media-picker-value"
28 on-iron-activate="onMediaPickerActivate_"> 18 on-iron-activate="onMediaPickerActivate_">
29 <template is="dom-repeat" items="[[devices]]"> 19 <template is="dom-repeat" items="[[devices]]">
30 <paper-item media-picker-value$="[[item.id]]"> 20 <div class="dropdown-item" media-picker-value$="[[item.id]]">
dpapad 2016/07/07 21:03:55 Let's add role="option" for accessibility.
stevenjb 2016/07/07 21:39:06 Done.
31 [[item.name]] 21 [[item.name]]
32 </paper-item> 22 </div>
33 </template> 23 </template>
34 </paper-listbox> 24 </paper-listbox>
35 </paper-dropdown-menu> 25 </paper-dropdown-menu>
36 </div> 26 </div>
37 </template> 27 </template>
38 <script src="media_picker.js"></script> 28 <script src="media_picker.js"></script>
39 </dom-module> 29 </dom-module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698