 Chromium Code Reviews
 Chromium Code Reviews Issue 1750873002:
  [MD settings] change ClearBrowsingData from a sub-page to a dialog  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1750873002:
  [MD settings] change ClearBrowsingData from a sub-page to a dialog  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| 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 * @fileoverview | 6 * @fileoverview | 
| 7 * Common dialog styles for Material Design settings. | 7 * Common dialog styles for Material Design settings. | 
| 8 */ | 8 */ | 
| 9 | 9 | 
| 10 <dom-module id="settings-dialog"> | 10 <dom-module id="settings-dialog"> | 
| 11 <template> | 11 <template> | 
| 12 <style> | 12 <style> | 
| 13 paper-dialog { | 13 paper-dialog { | 
| 14 max-width: 800px; | 14 max-width: 800px; | 
| 15 min-width: 500px; | 15 min-width: 500px; | 
| 16 } | 16 } | 
| 17 | 17 | 
| 18 #dialog-content { | 18 #dialog-content { | 
| 19 -webkit-padding-end: 0; | 19 -webkit-padding-end: 0; | 
| 20 -webkit-padding-start: 0; | 20 -webkit-padding-start: 0; | 
| 21 margin-bottom: 0; | 21 margin-bottom: 0; | 
| 22 margin-top: 0; | 22 margin-top: 0; | 
| 23 } | 23 } | 
| 24 | 24 | 
| 25 paper-dialog .top-row { | 25 paper-dialog .top-row { | 
| 26 align-items: center; | 26 align-items: center; | 
| 27 border-bottom: 1px solid gainsboro; | 27 border-bottom: 1px solid var(--paper-grey-300); | 
| 28 display: flex; | 28 display: flex; | 
| 29 padding-bottom: 5px; | 29 padding-bottom: 5px; | 
| 30 padding-top: 5px; | 30 padding-top: 5px; | 
| 31 } | 31 } | 
| 32 | 32 | 
| 33 paper-dialog .row { | |
| 34 align-items: center; | |
| 35 display: flex; | |
| 36 min-height: 40px; | |
| 37 } | |
| 38 | |
| 39 paper-dialog .row .start { | |
| 40 flex: 1; | |
| 41 } | |
| 42 | |
| 33 paper-dialog .title { | 43 paper-dialog .title { | 
| 34 flex: 1; | 44 flex: 1; | 
| 35 font-size: 1.13em; | 45 font-size: 1.13em; | 
| 36 } | 46 } | 
| 37 | 47 | 
| 38 paper-dialog .body { | 48 paper-dialog .body { | 
| 39 font-size: 1em; | 49 font-size: 1em; | 
| 40 margin: 20px 0; | 50 margin: 20px 0; | 
| 41 } | 51 } | 
| 42 | 52 | 
| 43 paper-dialog .title, | 53 paper-dialog .title, | 
| 44 paper-dialog .body { | 54 paper-dialog .body { | 
| 45 -webkit-padding-end: 24px; | 55 -webkit-padding-end: 24px; | 
| 46 -webkit-padding-start: 24px; | 56 -webkit-padding-start: 24px; | 
| 47 } | 57 } | 
| 48 | 58 | 
| 49 paper-dialog .cancel-button { | 59 paper-dialog .feedback-bar { | 
| 50 color: rgb(109, 109, 109); | 60 background-color: var(--paper-grey-200); | 
| 61 margin: 0; | |
| 62 padding: 20px; | |
| 63 } | |
| 64 | |
| 65 paper-dialog .action-button { | |
| 
Dan Beam
2016/03/04 00:30:42
why are you adding this back?  have you pulled lat
 
dschuyler
2016/03/04 02:27:16
I'll fix this after dpapad@ lands his dialog CL.
 | |
| 66 -webkit-margin-start: 10px; | |
| 51 font-weight: 500; | 67 font-weight: 500; | 
| 52 } | 68 } | 
| 53 | 69 | 
| 70 paper-dialog .action-button:not([disabled]) { | |
| 71 background-color: var(--google-blue-500); | |
| 72 color: white; | |
| 73 } | |
| 74 | |
| 75 paper-dialog .cancel-button { | |
| 76 color: var(--paper-grey-600); | |
| 77 font-weight: 500; | |
| 78 } | |
| 79 | |
| 54 paper-dialog .explanation { | 80 paper-dialog .explanation { | 
| 55 margin-bottom: 35px; | 81 margin-bottom: 35px; | 
| 56 } | 82 } | 
| 57 | 83 | 
| 58 paper-dialog .button-container { | 84 paper-dialog .button-container { | 
| 59 display: flex; | 85 display: flex; | 
| 60 justify-content: flex-end; | 86 justify-content: flex-end; | 
| 61 } | 87 } | 
| 62 | 88 | 
| 63 paper-dialog paper-button { | 89 paper-dialog paper-button { | 
| 64 margin: 0; | 90 margin: 0; | 
| 65 min-width: auto; | 91 min-width: auto; | 
| 66 } | 92 } | 
| 67 | 93 | 
| 68 paper-dialog paper-button[toggles][active] { | 94 paper-dialog paper-button[toggles][active] { | 
| 69 background-color: LightGray; | 95 background-color: var(--paper-grey-300); | 
| 70 } | 96 } | 
| 71 </style> | 97 </style> | 
| 72 </template> | 98 </template> | 
| 73 </dom-module> | 99 </dom-module> | 
| OLD | NEW |