Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* Copyright 2016 The Chromium Authors. All rights reserved. | 1 /* Copyright 2016 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 :root { | 6 :root { |
| 7 --fade-duration: 225ms; | |
| 7 --md-timing-function: cubic-bezier(.4, 0, .6, 1); | 8 --md-timing-function: cubic-bezier(.4, 0, .6, 1); |
| 8 --sidebar-width: 155px; | 9 --sidebar-width: 155px; |
| 9 --sidebar-neg-width: calc(var(--sidebar-width) * -1); | 10 --sidebar-neg-width: calc(var(--sidebar-width) * -1); |
| 10 } | 11 } |
| 11 | 12 |
| 12 html, | 13 html, |
| 13 body { | 14 body { |
| 14 margin: 0; | 15 margin: 0; |
| 15 padding: 0; | 16 padding: 0; |
| 16 } | 17 } |
| (...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 130 -webkit-border-start: 6px solid rgb(78, 87, 100); | 131 -webkit-border-start: 6px solid rgb(78, 87, 100); |
| 131 -webkit-padding-start: 10px; | 132 -webkit-padding-start: 10px; |
| 132 color: rgb(70, 78, 90); | 133 color: rgb(70, 78, 90); |
| 133 } | 134 } |
| 134 | 135 |
| 135 .sidebar-content button:hover { | 136 .sidebar-content button:hover { |
| 136 background-color: #E0E0E0; | 137 background-color: #E0E0E0; |
| 137 } | 138 } |
| 138 | 139 |
| 139 .overlay { | 140 .overlay { |
| 140 --fade-duration: 225ms; | |
| 141 background-color: rgba(0, 0, 0, .5); | 141 background-color: rgba(0, 0, 0, .5); |
| 142 bottom: 0; | 142 bottom: 0; |
| 143 left: 0; | 143 left: 0; |
| 144 opacity: 0; | 144 opacity: 0; |
| 145 position: absolute; | 145 position: absolute; |
| 146 right: 0; | 146 right: 0; |
| 147 top: 0; | 147 top: 0; |
| 148 transition: visibility var(--fade-duration), | 148 transition: visibility var(--fade-duration), |
| 149 opacity var(--fade-duration) var(--md-timing-function); | 149 opacity var(--fade-duration) var(--md-timing-function); |
| 150 visibility: hidden; | 150 visibility: hidden; |
| (...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 199 display: block; | 199 display: block; |
| 200 text-align: end; | 200 text-align: end; |
| 201 } | 201 } |
| 202 | 202 |
| 203 table td::before { | 203 table td::before { |
| 204 content: attr(data-label); | 204 content: attr(data-label); |
| 205 float: left; | 205 float: left; |
| 206 font-weight: bold; | 206 font-weight: bold; |
| 207 } | 207 } |
| 208 } | 208 } |
| 209 | |
| 210 /* Snackbar */ | |
| 211 .snackbar { | |
| 212 background-color: #323232; | |
| 213 border-radius: 2px; | |
| 214 bottom: 0; | |
| 215 color: #F1F1F1; | |
|
Dan Beam
2016/12/16 21:54:40
nit: html/css style guide generally prefers lower-
mbrunson
2016/12/17 02:21:58
Hmm ok. I fixed them all here. Done.
| |
| 216 display: flex; | |
| 217 font-size: 1.5em; | |
| 218 justify-content: center; | |
| 219 left: 0; | |
| 220 margin: 0 auto; | |
|
Dan Beam
2016/12/16 21:54:40
could we change this to margin: 12px; to position
mbrunson
2016/12/17 02:21:58
I was wondering why they have that margin. The mat
| |
| 221 max-width: 568px; | |
| 222 min-height: 20px; | |
| 223 min-width: 288px; | |
| 224 padding: 14px 24px; | |
| 225 position: fixed; | |
| 226 right: 0; | |
| 227 transform: translate3d(0, 48px, 0); | |
| 228 transition: transform var(--fade-duration), visibility var(--fade-duration); | |
| 229 transition-timing-function: var(--md-timing-function); | |
| 230 visibility: hidden; | |
| 231 } | |
| 232 | |
| 233 .snackbar a { | |
| 234 -webkit-margin-start: auto; | |
| 235 display: flex; | |
| 236 flex-direction: column; | |
| 237 justify-content: center; | |
| 238 text-transform: uppercase; | |
| 239 } | |
| 240 | |
| 241 @media screen and (max-width: 600px) { | |
| 242 .snackbar { | |
| 243 border-radius: 0; | |
| 244 max-width: unset; | |
| 245 min-width: unset; | |
|
Dan Beam
2016/12/16 21:54:40
rather than setting and unsetting, maybe just set
mbrunson
2016/12/17 02:21:59
Done.
| |
| 246 } | |
| 247 } | |
| 248 | |
| 249 .snackbar div { | |
| 250 align-self: flex-start; | |
| 251 } | |
| 252 | |
| 253 .snackbar a { | |
| 254 color: rgb(238, 255, 65); | |
| 255 } | |
| 256 | |
| 257 .snackbar.success { | |
| 258 background-color: rgb(76, 175, 80); | |
| 259 } | |
| 260 | |
| 261 .snackbar.warning { | |
| 262 background-color: rgb(255, 152, 0); | |
| 263 } | |
| 264 | |
| 265 .snackbar.warning a { | |
| 266 color: rgb(17, 85, 204); | |
| 267 } | |
| 268 | |
| 269 .snackbar.error { | |
| 270 background-color: rgb(244, 67, 54); | |
| 271 } | |
| 272 | |
| 273 .snackbar.open { | |
| 274 transform: translate3d(0, 0, 0); | |
|
Dan Beam
2016/12/16 21:54:40
if 2 snackbars show at once, does the more recent
mbrunson
2016/12/17 02:21:58
The most recent one would be on top since they're
| |
| 275 visibility: visible; | |
| 276 } | |
| OLD | NEW |