Chromium Code Reviews| Index: chrome/browser/resources/settings/settings_dialog.html |
| diff --git a/chrome/browser/resources/settings/settings_dialog.html b/chrome/browser/resources/settings/settings_dialog.html |
| index df72a0df54ef14921e24b7ac6c917da8f526e4ed..ff5f753865773bc58633751cf104debbbc064271 100644 |
| --- a/chrome/browser/resources/settings/settings_dialog.html |
| +++ b/chrome/browser/resources/settings/settings_dialog.html |
| @@ -9,8 +9,10 @@ |
| <style include="paper-dialog-shared-styles"></style> |
| <style> |
| .body-content { |
| + display: flex; |
| + flex-direction: column; |
| font-size: 1em; |
| - margin: 20px 0; |
| + min-height: 120px; |
| } |
| .dialog-content { |
| @@ -27,28 +29,38 @@ |
| .top-row { |
| align-items: center; |
| - border-bottom: 1px solid var(--paper-grey-300); |
| + border-bottom: 1px solid rgba(0, 0, 0, 0.14); |
| display: flex; |
| - padding-bottom: 5px; |
| - padding-top: 5px; |
| + font-size: 16px; |
|
dschuyler
2016/04/16 00:31:16
Please use a percentage of current font
size inste
dpapad
2016/04/16 01:09:51
Done. (Also moved the font-size style further down
|
| + height: 52px; |
|
dschuyler
2016/04/16 00:31:16
will min-height work here? I'm wondering
if a 'ver
dpapad
2016/04/16 01:09:51
Done, min-height seems to work just as fine.
|
| + } |
| + |
| + paper-icon-button { |
| + height: 20px; |
| + margin: 16px; |
| + padding: 0; |
| + width: 20px; |
| } |
| :host { |
| max-width: 800px; |
| - min-width: 500px; |
| + min-width: 512px; |
| overflow: auto; |
| } |
| :host ::content .body { |
| - margin-bottom: 35px; |
| + flex: 1; |
| + margin: 12px 0 24px 0; |
| } |
| :host ::content .button-container { |
| display: flex; |
| justify-content: flex-end; |
| + margin-bottom: 12px; |
| } |
| :host ::content .button-container .cancel-button { |
| + -webkit-margin-end: 8px; |
| color: var(--paper-grey-600); |
| font-weight: 500; |
| } |