From cf17daca99ba66c298af9b20967364c678e18e00 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fredrik=20Adel=C3=B6w?= Date: Thu, 19 Mar 2026 10:37:34 +0100 Subject: [PATCH] Only suppress loading for non-paginated tables MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit For paginated tables, always show the loading indicator since stale data from a different page would be misleading. The loading suppression (showing stale data during refresh) only applies to the non-paginated table where the stale filtered results are still meaningful. Co-Authored-By: Claude Opus 4.6 (1M context) Signed-off-by: Fredrik Adelöw --- .../src/components/CatalogTable/CatalogTable.tsx | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/plugins/catalog/src/components/CatalogTable/CatalogTable.tsx b/plugins/catalog/src/components/CatalogTable/CatalogTable.tsx index 093c92c1a0..80b37002c5 100644 --- a/plugins/catalog/src/components/CatalogTable/CatalogTable.tsx +++ b/plugins/catalog/src/components/CatalogTable/CatalogTable.tsx @@ -109,11 +109,13 @@ export const CatalogTable = (props: CatalogTableProps) => { paginationMode, } = entityListContext; - // Only show the full loading indicator when there's no data to display yet - // (i.e. initial load). During filter changes we already have stale data to - // show, so we keep it visible and let the new results swap in seamlessly - // instead of briefly flashing an empty table. - const isLoading = loading && entities.length === 0; + // For non-paginated tables, only show the full loading indicator when + // there's no data yet (initial load). During filter changes we keep stale + // data visible and let the new results swap in seamlessly. For paginated + // tables we always show loading, since stale data from a different page + // would be misleading. + const isLoading = + paginationMode === 'none' ? loading && entities.length === 0 : loading; const tableColumns = useMemo( () =>