fix(files): Do not jump to top when fileId is set to currentFolder

Fix #48545 Signed-off-by: Louis Chemineau <louis@chmn.me> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
nextcloud · Oct 29, 2024 · a77dd3e · a77dd3e
1 parent 749b6f3
commit a77dd3e
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/apps/files/src/components/FilesListVirtual.vue b/apps/files/src/components/FilesListVirtual.vue
@@ -268,6 +268,11 @@ export default defineComponent({
 
 		scrollToFile(fileId: number|null, warn = true) {
 			if (fileId) {
+				// Do not uselessly scroll to the top of the list.
+				if (fileId === this.currentFolder.fileid) {
+					return
+				}
+
 				const index = this.nodes.findIndex(node => node.fileid === fileId)
 				if (warn && index === -1 && fileId !== this.currentFolder.fileid) {
 					showError(this.t('files', 'File not found'))

diff --git a/dist/files-main.js b/dist/files-main.js
diff --git a/dist/files-main.js.map b/dist/files-main.js.map