Commit d9e6d0e3 authored by Luciano Pacheco's avatar Luciano Pacheco Committed by Commit Bot

Files app: Format dialog spacing and close button

Change the spacing between the last form field and the buttons in the
bottom. Reduce from 44px (24px from form field margin-bottom and 20px
from buttons padding-top). Add 2px padding-bottom for the field because
it was cutting the focus ring.

Remove the close icon by removing the attribute show-close-button in
<cr-dialog>.

This change affects both styles FilesNG and non-NG, since these are
minor, it shouldn't be a problem.

Bug: 992827
Change-Id: I8afc5beb92536d90ddbc007903d7d860edcc59cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2224741
Commit-Queue: Noel Gordon <noel@chromium.org>
Auto-Submit: Luciano Pacheco <lucmult@chromium.org>
Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#773923}
parent e63f921b
...@@ -26,6 +26,15 @@ ...@@ -26,6 +26,15 @@
margin-bottom: var(--cr-form-field-bottom-spacing); margin-bottom: var(--cr-form-field-bottom-spacing);
} }
[slot='body'] > #disk-format {
margin-bottom: 0;
padding-bottom: 2px;
}
[slot='button-container'] {
padding-top: 32px;
}
#warning-icon { #warning-icon {
--iron-icon-fill-color: var(--google-red-600); --iron-icon-fill-color: var(--google-red-600);
} }
...@@ -45,8 +54,7 @@ ...@@ -45,8 +54,7 @@
} }
</style> </style>
<cr-dialog id="dialog" show-close-button <cr-dialog id="dialog" close-text="[[i18n('CLOSE_LABEL')]]">
close-text="[[i18n('CLOSE_LABEL')]]" >
<div slot="title"> <div slot="title">
[[i18n('FORMAT_DIALOG_TITLE', volumeInfo_.label)]] [[i18n('FORMAT_DIALOG_TITLE', volumeInfo_.label)]]
</div> </div>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment