Skip to content

Commit f3a2806

Browse files
authored
Merge pull request #46681 from nextcloud/artonge/fix/grid_view_preview_size
fix(files): Reduce preview size in grid view mode
2 parents 55f81ea + d32ea63 commit f3a2806

File tree

4 files changed

+10
-12
lines changed

4 files changed

+10
-12
lines changed

apps/files/src/components/FilesListVirtual.vue

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -688,16 +688,15 @@ export default defineComponent({
688688
tbody.files-list__tbody.files-list__tbody--grid {
689689
--half-clickable-area: calc(var(--clickable-area) / 2);
690690
--item-padding: 16px;
691-
--icon-preview-size: 208px;
691+
--icon-preview-size: 166px;
692692
--name-height: 32px;
693693
--mtime-height: 16px;
694-
--row-width: calc(var(--icon-preview-size));
695-
--row-height: calc(var(--icon-preview-size) + var(--name-height) + var(--mtime-height));
694+
--row-width: calc(var(--icon-preview-size) + var(--item-padding) * 2);
695+
--row-height: calc(var(--icon-preview-size) + var(--name-height) + var(--mtime-height) + var(--item-padding) * 2);
696696
--checkbox-padding: 0px;
697697
698698
display: grid;
699699
grid-template-columns: repeat(auto-fill, var(--row-width));
700-
gap: 22px;
701700
702701
align-content: center;
703702
align-items: center;
@@ -714,7 +713,6 @@ tbody.files-list__tbody.files-list__tbody--grid {
714713
border: none;
715714
border-radius: var(--border-radius-large);
716715
padding: var(--item-padding);
717-
box-sizing: content-box;
718716
}
719717
720718
// Checkbox in the top left

apps/files/src/components/VirtualList.vue

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,13 +131,13 @@ export default Vue.extend({
131131
132132
itemHeight() {
133133
// Align with css in FilesListVirtual
134-
// 208px + 32px (name) + 16px (mtime) + 16px (padding) + 22px (grid gap)
135-
return this.gridMode ? (208 + 32 + 16 + 16 + 22) : 55
134+
// 166px + 32px (name) + 16px (mtime) + 16px (padding)
135+
return this.gridMode ? (166 + 32 + 16 + 16) : 55
136136
},
137137
// Grid mode only
138138
itemWidth() {
139-
// 208px + 16px padding + 22px grid gap
140-
return 208 + 16 + 22
139+
// 166px + 16px padding
140+
return 166 + 16
141141
},
142142
143143
rowCount() {

dist/files-main.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/files-main.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)