@acorex/components 22.1.0-next.16 → 22.1.0-next.18

package/fesm2022/acorex-components-data-table.mjs

@@ -2121,27 +2121,17 @@ class AXDataTableComponent extends AXBaseDataTable {
             this.applyDataPagerMethods();
         }
         this.rtl = AXHtmlUtil.isRtl(this.getHostElement());
-        this.dataSource.useCache = false;
-        if (this.fetchDataMode == 'auto') {
-            this.dataSource.setPage(0);
-        }
-        this.displayedRows.set(new Array(this.dataSource.pageSize));
-        this.totalRows = this.dataSource.items.length;
         this.dataSource.onLoadingChanged.pipe(this._unsubscriber.takeUntilDestroy).subscribe((data) => {
             this.isLoading.set(data);
         });
+        this.isLoading.set(this.dataSource.isLoading);
         this.dataSource.onChanged.pipe(this._unsubscriber.takeUntilDestroy).subscribe((data) => {
             const shouldMoveToPreviousPage = data.totalCount > 0 && data.items.length === 0 && data.page > 0;
             if (shouldMoveToPreviousPage) {
                 this.goToPage(data.page);
                 return;
             }
-            this.clearResolvedCssClasses();
-            this.lastLoadedRows = data.items.filter((item) => !item[this.parentField]);
-            this.displayedRows.set(this.mergeRowsWithHighlights(this.lastLoadedRows));
-            this.totalRows = data.totalCount;
-            this.hasItems = data.totalCount > 0;
-            this.scheduleTextHighlight();
+            this.applyLoadedRows(data.items, data.totalCount);
         });
         this.dataSource.onItemExpanded.pipe(this._unsubscriber.takeUntilDestroy).subscribe((data) => {
             const { expandedItem, children } = data;
@@ -2158,6 +2148,21 @@ class AXDataTableComponent extends AXBaseDataTable {
             this.refreshExpandedItem(id);
         });
         this.pageSize.set(this.dataSource.pageSize);
+        const existingItems = (this.dataSource.items ?? []).filter((item) => item != null);
+        if (existingItems.length > 0) {
+            // Popup/lookup remounts this table; setPage(0) is a no-op on a warm source and
+            // would leave the default skeleton rows in place. Paint cached rows instead of refetching.
+            this.applyLoadedRows(existingItems, this.dataSource.totalCount || existingItems.length);
+            this.isLoading.set(false);
+        }
+        else {
+            this.dataSource.useCache = false;
+            if (this.fetchDataMode == 'auto') {
+                this.dataSource.setPage(0);
+            }
+            this.displayedRows.set(new Array(this.dataSource.pageSize));
+            this.totalRows = this.dataSource.items.length;
+        }
         //
         this.clickSubject
             .pipe(buffer(this.clickSubject.pipe(debounceTime(250))), filter((clickArray) => clickArray.length === 2))
@@ -2165,6 +2170,14 @@ class AXDataTableComponent extends AXBaseDataTable {
             this.handleRowDoubleClick(clickArray[0].event, clickArray[0].item);
         });
     }
+    applyLoadedRows(items, totalCount) {
+        this.clearResolvedCssClasses();
+        this.lastLoadedRows = items.filter((item) => item != null && !item[this.parentField]);
+        this.displayedRows.set(this.mergeRowsWithHighlights(this.lastLoadedRows));
+        this.totalRows = totalCount;
+        this.hasItems = totalCount > 0;
+        this.scheduleTextHighlight();
+    }
     /**
      *  @ignore
      */