 Chromium Code Reviews
 Chromium Code Reviews Issue 1509913004:
  [MD settings] moving dialog css to shared to use in OnStartup and Reset settings  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1509913004:
  [MD settings] moving dialog css to shared to use in OnStartup and Reset settings  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW | 
|---|---|
| 1 /* Copyright 2015 The Chromium Authors. All rights reserved. | 1 /* Copyright 2015 The Chromium Authors. All rights reserved. | 
| 2 * Use of this source code is governed by a BSD-style license that can be | 2 * Use of this source code is governed by a BSD-style license that can be | 
| 3 * found in the LICENSE file. */ | 3 * found in the LICENSE file. */ | 
| 4 | 4 | 
| 5 /** | 5 /** | 
| 6 * @fileoverview | 6 * @fileoverview | 
| 7 * Common styles for Material Design settings. | 7 * Common styles for Material Design settings. | 
| 8 */ | 8 */ | 
| 9 | 9 | 
| 10 paper-button { | 10 /* List styles */ | 
| 11 margin: 0; | |
| 12 min-width: auto; | |
| 13 } | |
| 14 | |
| 15 paper-button[toggles][active] { | |
| 16 background-color: LightGray; | |
| 17 } | |
| 18 | |
| 19 h2 { | |
| 20 color: #9e9e9e; | |
| 21 font-size: 100%; | |
| 22 font-weight: normal; | |
| 23 margin-bottom: 20px; | |
| 24 margin-top: 30px; | |
| 25 } | |
| 26 | |
| 27 .button-row { | |
| 28 display: flex; | |
| 29 margin-top: 25px; | |
| 30 } | |
| 31 | |
| 32 .button-strip { | |
| 33 text-align: end; | |
| 34 } | |
| 35 | 11 | 
| 36 .list-frame { | 12 .list-frame { | 
| 37 @apply(--layout-center); | 13 @apply(--layout-center); | 
| 38 display: block; | 14 display: block; | 
| 39 margin-left: 48px; | 15 margin-left: 48px; | 
| 40 padding: 0; | 16 padding: 0; | 
| 41 } | 17 } | 
| 42 | 18 | 
| 43 .list-item { | 19 .list-item { | 
| 44 @apply(--layout-center); | 20 @apply(--layout-center); | 
| 45 border-top: 1px solid #e0e0e0; | 21 border-top: 1px solid #e0e0e0; | 
| 46 display: block; | 22 display: flex; | 
| 47 padding: 0; | 23 padding: 0; | 
| 48 } | 24 } | 
| 49 | 25 | 
| 50 .list-item:first-of-type { | 26 .list-item:first-of-type { | 
| 51 border-top: none; | 27 border-top: none; | 
| 52 } | 28 } | 
| 53 | 29 | 
| 30 .list-item .middle { | |
| 31 flex: 1; | |
| 32 margin-bottom: 8px; | |
| 33 margin-left: 12px; | |
| 34 margin-right: 12px; | |
| 35 margin-top: 8px; | |
| 36 overflow: hidden; | |
| 37 white-space: nowrap; | |
| 38 } | |
| 39 | |
| 54 .list-item > paper-icon-item { | 40 .list-item > paper-icon-item { | 
| 55 padding: 0; | 41 padding: 0; | 
| 56 } | 42 } | 
| 57 | 43 | 
| 58 .link-button { | 44 /* Section styles */ | 
| 59 color: rgb(66, 133, 244); | |
| 60 padding-bottom: 6px; | |
| 61 padding-left: 0; | |
| 62 padding-right: 0; | |
| 63 padding-top: 6px; | |
| 64 text-transform: none; | |
| 65 } | |
| 66 | 45 | 
| 67 .settings-box { | 46 .settings-box { | 
| 68 @apply(--layout-center); | 47 @apply(--layout-center); | 
| 69 border-top: 1px solid #e0e0e0; | 48 border-top: 1px solid #e0e0e0; | 
| 70 display: block; | 49 display: block; | 
| 71 min-height: 20px; | 50 min-height: 20px; | 
| 72 padding: 8px 16px; | 51 padding: 8px 16px; | 
| 73 } | 52 } | 
| 74 | 53 | 
| 75 .settings-box:first-of-type { | 54 .settings-box:first-of-type { | 
| 76 border-top: none; | 55 border-top: none; | 
| 77 } | 56 } | 
| 78 | 57 | 
| 79 .split { | 58 .split { | 
| 80 display: flex; | 59 display: flex; | 
| 81 } | 60 } | 
| 82 | 61 | 
| 83 .split .start { | 62 .split .start { | 
| 84 flex: auto; | 63 flex: auto; | 
| 85 } | 64 } | 
| 86 | 65 | 
| 87 .settings-box paper-item iron-icon { | 66 .settings-box paper-item iron-icon { | 
| 88 /* Same padding as paper-icon-button. */ | 67 /* Same padding as paper-icon-button. */ | 
| 89 padding: 8px; | 68 padding: 8px; | 
| 90 } | 69 } | 
| 70 | |
| 71 /* General styles */ | |
| 72 | |
| 73 .button-strip { | |
| 74 text-align: end; | |
| 75 } | |
| 76 | |
| 77 .link-button { | |
| 
dpapad
2015/12/09 23:12:06
Are we still using buttons as links? They were rem
 
dschuyler
2015/12/09 23:29:59
Acknowledged.
 | |
| 78 color: rgb(66, 133, 244); | |
| 79 padding-bottom: 6px; | |
| 80 padding-left: 0; | |
| 81 padding-right: 0; | |
| 82 padding-top: 6px; | |
| 
dpapad
2015/12/09 23:12:06
Lines 79-82 can be compressed as follows:
padding:
 
dschuyler
2015/12/09 23:29:59
Acknowledged.
 | |
| 83 text-transform: none; | |
| 84 } | |
| 85 | |
| 86 .text-elide { | |
| 87 overflow: hidden; | |
| 88 text-overflow: ellipsis; | |
| 89 white-space: nowrap; | |
| 90 } | |
| 91 | |
| 92 .secondary-color { | |
| 
dpapad
2015/12/09 23:12:06
This one is a bit odd. 
1) Secondary to what? Is t
 
dschuyler
2015/12/09 23:30:00
Acknowledged.
 | |
| 93 color: #969696; | |
| 94 } | |
| 95 | |
| 96 h2 { | |
| 97 color: #9e9e9e; | |
| 98 font-size: 100%; | |
| 99 font-weight: normal; | |
| 100 margin-bottom: 20px; | |
| 101 margin-top: 30px; | |
| 102 } | |
| OLD | NEW |