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

Side by Side Diff: chrome/browser/resources/settings/settings_shared_css.html

Issue 2143733002: MD Settings: settings-dropdown-menu, restore "Enter" key behavior. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/cr_elements/shared_style_css.html"> 1 <link rel="import" href="chrome://resources/cr_elements/shared_style_css.html">
2 <link rel="import" href="/settings_root_css.html"> 2 <link rel="import" href="/settings_root_css.html">
3 3
4 <!-- Common styles for Material Design settings. --> 4 <!-- Common styles for Material Design settings. -->
5 <dom-module id="settings-shared"> 5 <dom-module id="settings-shared">
6 <template> 6 <template>
7 <style include="cr-shared-style"> 7 <style include="cr-shared-style">
8 h2 { 8 h2 {
9 align-items: center; 9 align-items: center;
10 display: flex; 10 display: flex;
(...skipping 21 matching lines...) Expand all
32 --paper-font-subhead: { 32 --paper-font-subhead: {
33 font-size: inherit; 33 font-size: inherit;
34 }; 34 };
35 --paper-input-container-underline: { 35 --paper-input-container-underline: {
36 background: var(--paper-grey-300); 36 background: var(--paper-grey-300);
37 }; 37 };
38 } 38 }
39 39
40 paper-dropdown-menu .dropdown-item { 40 paper-dropdown-menu .dropdown-item {
41 align-items: center; 41 align-items: center;
42 background: none;
43 border: none;
Dan Beam 2016/07/12 23:00:27 nit: -webkit-appearance: none;
dpapad 2016/07/12 23:18:02 Does not seem to work equivalently for me, see htt
Dan Beam 2016/07/12 23:27:26 we use it in widgets.css and otherwise a -webkit-a
42 color: var(--paper-grey-800); 44 color: var(--paper-grey-800);
43 display: flex; 45 display: flex;
44 font-size: inherit; 46 font-size: inherit;
Dan Beam 2016/07/12 23:00:27 do we need to change this to font: inherit? i rem
dpapad 2016/07/12 23:18:02 Done. Changing 'font-size' to 'font' indeed has an
45 min-height: 48px; 47 min-height: 48px;
46 padding: 0 16px; 48 padding: 0 16px;
49 width: 100%;
47 } 50 }
48 51
49 paper-dropdown-menu .dropdown-item.iron-selected { 52 paper-dropdown-menu .dropdown-item.iron-selected {
50 font-weight: bold; 53 font-weight: bold;
51 } 54 }
52 55
53 paper-dropdown-menu .dropdown-item:focus { 56 paper-dropdown-menu .dropdown-item:focus {
54 background-color: var(--paper-grey-300); 57 background-color: var(--paper-grey-300);
55 outline: none; 58 outline: none;
56 } 59 }
(...skipping 264 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 324
322 .favicon-image { 325 .favicon-image {
323 background-repeat: no-repeat; 326 background-repeat: no-repeat;
324 background-size: contain; 327 background-size: contain;
325 height: 16px; 328 height: 16px;
326 width: 16px; 329 width: 16px;
327 } 330 }
328 </style> 331 </style>
329 </template> 332 </template>
330 </dom-module> 333 </dom-module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698