@tap-payments/os-micro-frontend-shared 0.1.136-test.2 → 0.1.136-test.4

package/build/components/VirtualTables/SheetViewVirtualTable/components/SheetViewTableHeaderCell.js

@@ -51,6 +51,6 @@ function SheetViewTableHeaderCell({ column: { header, id, align, headerStyle, so
                     } })), isPinnable && pinnable && !isDefaultPinned && !isLast && (_jsx(PinIconContainer, Object.assign({ onClick: (event) => {
                         event.stopPropagation();
                         handlePinClick(id);
-                    }, title: isPinned ? 'Unpin column' : 'Pin column' }, { children: _jsx("img", { src: isPinned ? unpinIcon : pinIcon, alt: "pin-icon" }) })))] })) })));
+                    }, title: isPinned ? 'Unpin column' : 'Pin column' }, { children: _jsx("img", { src: isPinned ? unpinIcon : pinIcon, alt: "pin-icon" }) })))] })) }), `${id}-${colIndex}`));
 }
 export default memo(SheetViewTableHeaderCell);

package/build/components/VirtualTables/SheetViewVirtualTable/hooks/usePerformanceOptimizations.js

@@ -1,88 +1,22 @@
 import { useMemo, useRef } from 'react';
-// Deep comparison helper for filter data
-const areFilterDataEqual = (data1, data2) => {
-    if (data1 === data2)
-        return true;
-    if (!data1 || !data2)
-        return false;
-    const keys1 = Object.keys(data1);
-    const keys2 = Object.keys(data2);
-    if (keys1.length !== keys2.length)
-        return false;
-    return keys1.every((key) => {
-        const val1 = data1[key];
-        const val2 = data2[key];
-        if (Array.isArray(val1) && Array.isArray(val2)) {
-            return val1.length === val2.length && val1.every((v, i) => v === val2[i]);
-        }
-        if (typeof val1 === 'object' && typeof val2 === 'object' && val1 && val2) {
-            return JSON.stringify(val1) === JSON.stringify(val2);
-        }
-        return val1 === val2;
-    });
-};
 export const usePerformanceOptimizations = (columns) => {
     // Create stable column references to prevent unnecessary re-renders
     const stableColumnsRef = useRef([]);
-    const isFirstRenderRef = useRef(true);
-    const filterChangeTrackingRef = useRef(new Map());
     const stableColumns = useMemo(() => {
-        // On first render, just initialize and return columns
-        if (isFirstRenderRef.current) {
-            isFirstRenderRef.current = false;
-            stableColumnsRef.current = [...columns];
-            // Initialize filter tracking
-            columns.forEach((col) => {
-                var _a;
-                filterChangeTrackingRef.current.set(col.id, (_a = col.filter) === null || _a === void 0 ? void 0 : _a.data);
-            });
-            return stableColumnsRef.current;
-        }
-        // After first render, only update if structural changes or filter changes occurred
-        const structurallyChanged = columns.length !== stableColumnsRef.current.length ||
+        // Only update if columns have actually changed (deep comparison of essential properties)
+        const hasChanged = columns.length !== stableColumnsRef.current.length ||
             columns.some((col, index) => {
                 const prevCol = stableColumnsRef.current[index];
-                console.log({ colFilter: col.filter, prevColFilter: prevCol === null || prevCol === void 0 ? void 0 : prevCol.filter });
                 return (!prevCol ||
                     col.id !== prevCol.id ||
                     col.width !== prevCol.width ||
                     col.pinned !== prevCol.pinned ||
                     col.hidden !== prevCol.hidden ||
-                    col.order !== prevCol.order);
+                    col.order !== prevCol.order ||
+                    col.filter !== prevCol.filter);
             });
-        // Check for filter-only changes
-        const columnsWithFilterChanges = new Set();
-        columns.forEach((col) => {
-            var _a;
-            const colId = col.id;
-            const prevFilterData = filterChangeTrackingRef.current.get(colId);
-            const currentFilterData = (_a = col.filter) === null || _a === void 0 ? void 0 : _a.data;
-            if (!areFilterDataEqual(prevFilterData, currentFilterData)) {
-                columnsWithFilterChanges.add(colId);
-                filterChangeTrackingRef.current.set(colId, currentFilterData);
-            }
-        });
-        // Only update if there are structural changes or filter changes
-        if (structurallyChanged || columnsWithFilterChanges.size > 0) {
-            // For filter-only changes, update only the affected columns
-            if (!structurallyChanged && columnsWithFilterChanges.size > 0) {
-                // Create a new array but preserve references for unchanged columns
-                const updatedColumns = stableColumnsRef.current.map((stableCol, index) => {
-                    const newCol = columns[index];
-                    const colId = newCol === null || newCol === void 0 ? void 0 : newCol.id;
-                    // Only replace if this column's filter changed
-                    if (columnsWithFilterChanges.has(colId)) {
-                        return newCol;
-                    }
-                    // Keep the stable reference for unchanged columns
-                    return stableCol;
-                });
-                stableColumnsRef.current = updatedColumns;
-            }
-            else {
-                // Full update for structural changes
-                stableColumnsRef.current = [...columns];
-            }
+        if (hasChanged) {
+            stableColumnsRef.current = [...columns];
         }
         return stableColumnsRef.current;
     }, [columns]);

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@tap-payments/os-micro-frontend-shared",
   "description": "Shared components and utilities for Tap Payments micro frontends",
-  "version": "0.1.136-test.2",
-  "testVersion": 2,
+  "version": "0.1.136-test.4",
+  "testVersion": 4,
   "type": "module",
   "main": "build/index.js",
   "module": "build/index.js",