From c495ba30f8b1bff3fed342565e3ecffc80c01397 Mon Sep 17 00:00:00 2001 From: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> Date: Thu, 26 Feb 2026 19:00:23 -0300 Subject: [PATCH 1/2] fix: use same grid/list view icons as files app Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> --- src/views/FilesList/FilesList.vue | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/views/FilesList/FilesList.vue b/src/views/FilesList/FilesList.vue index 3bdb239ebd..a457a49444 100644 --- a/src/views/FilesList/FilesList.vue +++ b/src/views/FilesList/FilesList.vue @@ -49,8 +49,8 @@ variant="tertiary" @click="toggleGridView"> @@ -96,9 +96,9 @@ import { mdiChevronDown, mdiChevronUp, mdiFolder, + mdiFormatListBulletedSquare, mdiReload, - mdiViewGrid, - mdiViewList, + mdiViewGridOutline, } from '@mdi/js' import NcActionButton from '@nextcloud/vue/components/NcActionButton' @@ -147,9 +147,9 @@ export default { mdiChevronDown, mdiChevronUp, mdiFolder, + mdiFormatListBulletedSquare, mdiReload, - mdiViewGrid, - mdiViewList, + mdiViewGridOutline, } }, data() { From 901ec42bb92f7a61dadd9e527b4bc46d247fdb91 Mon Sep 17 00:00:00 2001 From: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> Date: Thu, 26 Feb 2026 19:15:27 -0300 Subject: [PATCH 2/2] test(FilesList): fix mdi icon names to match component Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> --- src/tests/views/FilesList/FilesList.spec.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/tests/views/FilesList/FilesList.spec.ts b/src/tests/views/FilesList/FilesList.spec.ts index ed4f8b5336..caa08445d5 100644 --- a/src/tests/views/FilesList/FilesList.spec.ts +++ b/src/tests/views/FilesList/FilesList.spec.ts @@ -156,8 +156,8 @@ describe('FilesList.vue rendering rules', () => { await flushPromises() expect(wrapper.vm.mdiFolder).toBeTruthy() - expect(wrapper.vm.mdiViewGrid).toBeTruthy() - expect(wrapper.vm.mdiViewList).toBeTruthy() + expect(wrapper.vm.mdiViewGridOutline).toBeTruthy() + expect(wrapper.vm.mdiFormatListBulletedSquare).toBeTruthy() expect(wrapper.vm.mdiChevronDown).toBeTruthy() expect(wrapper.vm.mdiChevronUp).toBeTruthy() expect(wrapper.vm.mdiReload).toBeTruthy() @@ -265,7 +265,7 @@ describe('FilesList.vue rendering rules', () => { const gridButton = wrapper.find('.files-list__header-grid-button') const iconWithPath = gridButton.findAll('.nc-icon').find((node) => !!node.attributes('data-path')) - expect(iconWithPath?.attributes('data-path')).toBe(wrapper.vm.mdiViewGrid) + expect(iconWithPath?.attributes('data-path')).toBe(wrapper.vm.mdiViewGridOutline) }) it('renders list toggle icon path when in grid mode', async () => { @@ -279,6 +279,6 @@ describe('FilesList.vue rendering rules', () => { const gridButton = wrapper.find('.files-list__header-grid-button') const iconWithPath = gridButton.findAll('.nc-icon').find((node) => !!node.attributes('data-path')) - expect(iconWithPath?.attributes('data-path')).toBe(wrapper.vm.mdiViewList) + expect(iconWithPath?.attributes('data-path')).toBe(wrapper.vm.mdiFormatListBulletedSquare) }) })