Chromium Code Reviews Chromium Code Reviews
Issue 1766473003:
Sync confirmation: UI tweaks from bettes@ (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@dschuyler-patch| 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 body { | 5 body { |
| 6 font-size: 100%; | |
|
Dan Beam
2016/03/04 02:12:38
instead of making the bigger font-size the default
| |
| 7 margin: 0; | 6 margin: 0; |
| 8 padding: 0; | 7 padding: 0; |
| 9 } | 8 } |
| 10 | 9 |
| 11 a { | 10 a { |
| 12 color: rgb(85, 149, 254); | 11 color: rgb(85, 149, 254); |
| 13 text-decoration: none; | 12 text-decoration: none; |
| 14 } | 13 } |
| 15 | 14 |
| 16 .picture img { | 15 .picture img { |
| 17 border-radius: 50%; | 16 border-radius: 50%; |
| 18 max-height: 100%; | 17 max-height: 100%; |
| 19 max-width: 100%; | 18 max-width: 100%; |
| 20 } | 19 } |
| 21 | 20 |
| 22 .container { | 21 .container { |
| 23 background-color: white; | 22 background-color: white; |
| 24 height: 351px; | 23 height: 351px; |
|
Dan Beam
2016/03/04 02:12:38
i can update this and the corresponding kSyncConfi
anthonyvd
2016/03/04 15:04:47
Not a fan either but I wasn't able to find any way
Dan Beam
2016/03/04 20:11:10
we could measure the scrollHeight of the <body> an
| |
| 25 overflow: hidden; | 24 overflow: hidden; |
| 26 width: 448px; | 25 width: 448px; |
| 27 } | 26 } |
| 28 | 27 |
| 29 .top-title-bar { | 28 .top-title-bar { |
| 30 -webkit-padding-start: 24px; | 29 -webkit-padding-start: 24px; |
| 31 align-items: center; | 30 align-items: center; |
| 32 border-bottom: 1px solid lightgray; | 31 border-bottom: 1px solid lightgray; |
| 33 color: #333; | 32 color: #333; |
| 34 display: flex; | 33 display: flex; |
| 35 font-size: 1em; | 34 font-size: 16px; |
|
Dan Beam
2016/03/04 02:12:38
i originally wanted to make this and the below lin
anthonyvd
2016/03/04 15:04:47
It used to and it actually came up during UI revie
| |
| 36 height: 56px; | 35 height: 52px; |
| 37 } | 36 } |
| 38 | 37 |
| 39 .details { | 38 .details { |
| 40 height: 250px; | 39 height: 250px; |
| 41 padding: 20px 24px; | 40 padding: 20px 24px; |
| 42 } | 41 } |
| 43 | 42 |
| 44 .sync-message { | 43 .sync-message { |
| 45 color: #595959; | 44 color: #595959; |
| 46 font-size: 0.8125em; | |
| 47 line-height: 150%; | 45 line-height: 150%; |
| 48 padding-bottom: 32px; | 46 padding-bottom: 32px; |
| 49 } | 47 } |
| 50 | 48 |
| 51 #picture-container { | 49 #picture-container { |
| 52 align-items: center; | 50 align-items: center; |
| 53 display: flex; | 51 display: flex; |
| 54 justify-content: center; | 52 justify-content: center; |
| 55 padding-bottom: 32px; | 53 padding-bottom: 32px; |
| 56 } | 54 } |
| (...skipping 10 matching lines...) Expand all Loading... | |
| 67 position: absolute; | 65 position: absolute; |
| 68 } | 66 } |
| 69 | 67 |
| 70 .action-container { | 68 .action-container { |
| 71 align-items: baseline; | 69 align-items: baseline; |
| 72 display: flex; | 70 display: flex; |
| 73 justify-content: flex-end; | 71 justify-content: flex-end; |
| 74 } | 72 } |
| 75 | 73 |
| 76 paper-button { | 74 paper-button { |
| 77 font-size: 0.8125em; | 75 font-weight: 500; |
| 78 padding-left: 12px; | 76 line-height: 16px; |
| 79 padding-right: 12px; | 77 margin: 0; |
| 78 padding: 8px 16px; | |
| 79 } | |
| 80 | |
| 81 paper-button + paper-button { | |
| 82 -webkit-margin-start: 8px; | |
| 80 } | 83 } |
| 81 | 84 |
| 82 #confirmButton { | 85 #confirmButton { |
| 83 background-color: rgb(66, 133, 244); | 86 background-color: rgb(66, 133, 244); |
| 84 color: white; | 87 color: white; |
| 85 } | 88 } |
| 86 | 89 |
| 87 #undoButton { | 90 #undoButton { |
| 88 color: #5A5A5A; | 91 color: #5A5A5A; |
| 89 } | 92 } |
| (...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 314 @keyframes fade-in-icon-bottom-right { | 317 @keyframes fade-in-icon-bottom-right { |
| 315 from { | 318 from { |
| 316 opacity: 0; | 319 opacity: 0; |
| 317 transform: translate(0, 0); | 320 transform: translate(0, 0); |
| 318 } | 321 } |
| 319 to { | 322 to { |
| 320 opacity: 0.1; | 323 opacity: 0.1; |
| 321 transform: translate(4px, 4px); | 324 transform: translate(4px, 4px); |
| 322 } | 325 } |
| 323 } | 326 } |
| OLD | NEW |
