Chromium Code Reviews| Index: ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html |
| diff --git a/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html b/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html |
| index 82057a43d1b544fe1b3737545093ff793e4f57f6..466b5a05b204aae6699d6ea77b2181b6f916925d 100644 |
| --- a/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html |
| +++ b/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html |
| @@ -1,25 +1,11 @@ |
| <link rel="import" href="chrome://resources/cr_elements/icons.html"> |
| <link rel="import" href="chrome://resources/html/assert.html"> |
| <link rel="import" href="chrome://resources/html/polymer.html"> |
| -<link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/paper-dialog-behavior.html"> |
| -<link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/paper-dialog-shared-styles.html"> |
| <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html"> |
| <dom-module id="cr-dialog"> |
| <template> |
| - <style include="paper-dialog-shared-styles"> |
| - :host { |
| - border-radius: 2px; |
| - } |
| - |
| - :host ::content > * { |
| - /* Overrides paper-dialog-shared-styles. */ |
| - -webkit-padding-end: 0; |
| - -webkit-padding-start: 0; |
| - margin-bottom: 0; |
| - margin-top: 0; |
| - } |
| - |
| + <style> |
| .title-container { |
| align-items: center; |
| /* TODO(dbeam): should this be a --settings-separator-line? */ |
| @@ -29,7 +15,7 @@ |
| } |
| :host ::content .title { |
| - font-size: 114.28%; /* (16px / 14px) * 100 */ |
| + font-size: 123.07%; /* (16px / 13px) * 100 */ |
| } |
| #close { |
| @@ -63,12 +49,6 @@ |
| flex: 1; |
| } |
| - :host ::content .body, |
| - :host ::content .footer, |
| - :host ::content paper-button { |
| - font-size: 92.86%; /* (13px / 14px) * 100 */ |
| - } |
| - |
| :host ::content .button-container { |
| -webkit-padding-end: 16px; |
| -webkit-padding-start: 16px; |
| @@ -90,6 +70,24 @@ |
| margin: 0; |
| padding: 12px 20px; |
| } |
| + |
| + :host { |
| + border: 0; |
| + border-radius: 2px; |
| + bottom: 0; |
| + color: inherit; |
| + padding: 0; |
| + top: 0; |
| + } |
| + |
| + :host::backdrop { |
| + background-color: rgba(0, 0, 0, 0.6); |
| + bottom: 0; |
| + left: 0; |
| + position: fixed; |
| + right: 0; |
| + top: 0; |
| + } |
|
Dan Beam
2016/07/26 23:36:12
i would argue that these should go on the top of t
dpapad
2016/07/27 01:21:47
Done.
|
| </style> |
| <div class="title-container"> |
| <content select=".title"></content> |