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

Files app: File list FilesNG

Table header move padding to the label so clicking in the padding area
activates the click event in the label to sort the column.

Change the cursor to pointer in the column header to indicate that it
has an action (sorting the column).

Reset the table header font-weight because it's already using Robot
Medium which is already the right weight.

Fix the border radius for the file row thumbnail so it shows perfectly
rounded.

Reset the padding top in the file row because in the current spec the
text doesn't need the additional padding to align with the icon [1].

[1] -  https://cs.chromium.org/chromium/src/ui/file_manager/file_manager/foreground/css/common.css?l=343&rcl=80e9555e489ff4a6743f2d8f99dafdbaccc7397f

Bug: 992821
Change-Id: I06b1b6e5c842162f6b1a3ea027932869704995a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2033059
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@{#738513}
parent aaa6ca07
...@@ -2189,16 +2189,21 @@ body.files-ng .table-header { ...@@ -2189,16 +2189,21 @@ body.files-ng .table-header {
font-family: 'Roboto Medium'; font-family: 'Roboto Medium';
height: 56px; height: 56px;
line-height: 20px; line-height: 20px;
padding-bottom: 16px;
padding-top: 20px;
} }
body.files-ng .table-header-label { body.files-ng .table-header-label {
font-weight: normal;
line-height: 20px; line-height: 20px;
margin-inline-end: 5px; margin-inline-end: 5px;
margin-inline-start: 24px; margin-inline-start: 24px;
} }
body.files-ng .table-header-cell {
cursor: pointer;
padding-bottom: 16px;
padding-top: 20px;
}
body.files-ng #list-container .table-header-cell:first-child body.files-ng #list-container .table-header-cell:first-child
.table-header-label { .table-header-label {
margin-inline-start: 12px; margin-inline-start: 12px;
...@@ -2244,10 +2249,15 @@ body.files-ng.check-select #list-container list li[selected] .detail-thumbnail ...@@ -2244,10 +2249,15 @@ body.files-ng.check-select #list-container list li[selected] .detail-thumbnail
animation: none; animation: none;
} }
body.files-ng #list-container list li .detail-thumbnail > .thumbnail {
border-radius: 16px;
}
body.files-ng .table-row-cell > * { body.files-ng .table-row-cell > * {
color: var(--google-grey-700); color: var(--google-grey-700);
padding: 0; padding: 0;
padding-inline-start: 24px; padding-inline-start: 24px;
padding-top: 0 !important;
} }
body.files-ng #list-container .table-row-cell .size { body.files-ng #list-container .table-row-cell .size {
...@@ -2265,6 +2275,7 @@ body.files-ng .table-row-cell:first-child > * { ...@@ -2265,6 +2275,7 @@ body.files-ng .table-row-cell:first-child > * {
body.files-ng .table-row-cell .filename-label { body.files-ng .table-row-cell .filename-label {
color: var(--google-grey-900); color: var(--google-grey-900);
padding-top: 0;
} }
body.files-ng.check-select list:focus li[selected] .table-row-cell body.files-ng.check-select list:focus li[selected] .table-row-cell
......
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