Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 <link rel="import" href="chrome://resources/cr_elements/icons.html"> | 1 <link rel="import" href="chrome://resources/cr_elements/icons.html"> |
| 2 <link rel="import" href="chrome://resources/html/assert.html"> | 2 <link rel="import" href="chrome://resources/html/assert.html"> |
| 3 <link rel="import" href="chrome://resources/html/polymer.html"> | 3 <link rel="import" href="chrome://resources/html/polymer.html"> |
| 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/p aper-dialog-behavior.html"> | 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/p aper-dialog-behavior.html"> |
| 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/p aper-dialog-shared-styles.html"> | 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/p aper-dialog-shared-styles.html"> |
| 6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button.html"> | 6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button.html"> |
| 7 | 7 |
| 8 <dom-module id="cr-dialog"> | 8 <dom-module id="cr-dialog"> |
| 9 <template> | 9 <template> |
| 10 <style include="paper-dialog-shared-styles"> | 10 <style include="paper-dialog-shared-styles"> |
| 11 :host { | 11 :host { |
| 12 border-radius: 2px; | 12 border-radius: 2px; |
| 13 } | 13 } |
| 14 | 14 |
| 15 :host ::content > * { | 15 :host ::content > * { |
| 16 /* Overrides paper-dialog-shared-styles. */ | 16 /* Overrides paper-dialog-shared-styles. */ |
| 17 -webkit-padding-end: 0; | 17 -webkit-padding-end: 0; |
| 18 -webkit-padding-start: 0; | 18 -webkit-padding-start: 0; |
| 19 margin-bottom: 0; | 19 margin-bottom: 0; |
| 20 margin-top: 0; | 20 margin-top: 0; |
| 21 } | 21 } |
| 22 | 22 |
| 23 .title-container { | 23 .title-container { |
| 24 align-items: center; | 24 align-items: center; |
| 25 /* TODO(dbeam): should this be a --settings-separator-line? */ | 25 /* TODO(dbeam): should this be a --settings-separator-line? */ |
|
Dan Beam
2016/06/28 20:40:25
this TODO was written when this code lived in:
chr
msramek
2016/06/29 11:32:22
Acknowledged.
| |
| 26 border-bottom: 1px solid rgba(0, 0, 0, 0.14); | 26 border-bottom: var(--settings-dialog-title-border); |
|
Dan Beam
2016/06/28 20:39:44
this should not reference settings
msramek
2016/06/29 11:32:22
Makes sense. I'll just use the exact color for now
| |
| 27 display: flex; | 27 display: flex; |
| 28 min-height: 52px; | 28 min-height: 52px; |
| 29 } | 29 } |
| 30 | 30 |
| 31 :host ::content .title { | 31 :host ::content .title { |
| 32 font-size: 114.28%; /* (16px / 14px) * 100 */ | 32 font-size: 114.28%; /* (16px / 14px) * 100 */ |
| 33 } | 33 } |
| 34 | 34 |
| 35 #close { | 35 #close { |
| 36 --paper-icon-button: { | 36 --paper-icon-button: { |
| (...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 97 </paper-icon-button> | 97 </paper-icon-button> |
| 98 </div> | 98 </div> |
| 99 <div class="body-container"> | 99 <div class="body-container"> |
| 100 <content select=".body"></content> | 100 <content select=".body"></content> |
| 101 </div> | 101 </div> |
| 102 <content select=".button-container"></content> | 102 <content select=".button-container"></content> |
| 103 <content select=".footer"></content> | 103 <content select=".footer"></content> |
| 104 </template> | 104 </template> |
| 105 <script src="cr_dialog.js"></script> | 105 <script src="cr_dialog.js"></script> |
| 106 </dom-module> | 106 </dom-module> |
| OLD | NEW |