Merge pull request #18395 from davelopez/24.1_fix_filesources_subfold…

…er_pagination [24.1] Reset current page when browsing sub-folders in FilesSources
galaxyproject · Jun 14, 2024 · f2de606 · f2de606
2 parents ab533b6 + 03001d4
commit f2de606
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/client/src/components/FilesDialog/FilesDialog.vue b/client/src/components/FilesDialog/FilesDialog.vue
@@ -409,6 +409,7 @@ onMounted(() => {
         :is-busy="isBusy"
         :items="items"
         :items-provider="itemsProvider"
+        :provider-url="currentDirectory?.url"
         :total-items="totalItems"
         :modal-show="modalShow"
         :modal-static="modalStatic"

diff --git a/client/src/components/SelectionDialog/SelectionDialog.vue b/client/src/components/SelectionDialog/SelectionDialog.vue
@@ -27,6 +27,7 @@ interface Props {
     isEncoded?: boolean;
     items?: SelectionItem[];
     itemsProvider?: ItemsProvider;
+    providerUrl?: string;
     totalItems?: number;
     leafIcon?: string;
     modalShow?: boolean;
@@ -48,6 +49,7 @@ const props = withDefaults(defineProps<Props>(), {
     isEncoded: false,
     items: () => [],
     itemsProvider: undefined,
+    providerUrl: undefined,
     totalItems: 0,
     leafIcon: "fa fa-file-o",
     modalShow: true,
@@ -127,6 +129,16 @@ watch(
         filtered(props.items);
     }
 );
+
+watch(
+    () => props.providerUrl,
+    () => {
+        // We need to reset the current page when drilling down sub-folders
+        if (props.itemsProvider !== undefined) {
+            currentPage.value = 1;
+        }
+    }
+);
 </script>
 
 <template>