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

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

Issue 2133793003: Unify iron-dropdown CSS (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 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 48
49 paper-dropdown-menu .dropdown-item.iron-selected { 49 paper-dropdown-menu .dropdown-item.iron-selected {
50 font-weight: bold; 50 font-weight: bold;
51 } 51 }
52 52
53 paper-dropdown-menu .dropdown-item:focus { 53 paper-dropdown-menu .dropdown-item:focus {
54 background-color: var(--paper-grey-300); 54 background-color: var(--paper-grey-300);
55 outline: none; 55 outline: none;
56 } 56 }
57 57
58 iron-dropdown .dropdown-content {
59 background-color: white;
60 box-shadow: 0 2px 6px var(--paper-grey-500);
61 }
62
63 iron-dropdown .dropdown-content paper-item:hover {
dpapad 2016/07/08 21:01:29 Do we want to preserve the hover effect? In dropdo
stevenjb 2016/07/08 21:03:53 I'd prefer to make both changes in a separate clea
64 @apply(--settings-actionable);
Dan Beam 2016/07/08 23:08:08 I probably would've done this: iron-dropdown .dro
stevenjb 2016/07/08 23:50:05 Oh, yeah, that's actually what I had in my head. I
65 background-color: var(--settings-hover-color);
66 }
67
58 span ~ a { 68 span ~ a {
59 -webkit-margin-start: 4px; 69 -webkit-margin-start: 4px;
60 } 70 }
61 71
62 [is='action-link'], 72 [is='action-link'],
63 [is='action-link']:active, 73 [is='action-link']:active,
64 [is='action-link']:hover, 74 [is='action-link']:hover,
65 [is='action-link']:visited, 75 [is='action-link']:visited,
66 paper-button.primary-button, 76 paper-button.primary-button,
67 paper-button.tertiary-button { 77 paper-button.tertiary-button {
(...skipping 253 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 331
322 .favicon-image { 332 .favicon-image {
323 background-repeat: no-repeat; 333 background-repeat: no-repeat;
324 background-size: contain; 334 background-size: contain;
325 height: 16px; 335 height: 16px;
326 width: 16px; 336 width: 16px;
327 } 337 }
328 </style> 338 </style>
329 </template> 339 </template>
330 </dom-module> 340 </dom-module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698