@alaarab/ogrid-angular 2.0.4 → 2.0.5

package/dist/esm/components/grid-context-menu.component.js

@@ -4,7 +4,7 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
-import { Component, input, output, effect, viewChild, DestroyRef, inject } from '@angular/core';
+import { Component, input, output, viewChild, DestroyRef, inject } from '@angular/core';
 import { CommonModule } from '@angular/common';
 import { GRID_CONTEXT_MENU_ITEMS, formatShortcut } from '@alaarab/ogrid-core';
 let GridContextMenuComponent = class GridContextMenuComponent {
@@ -35,11 +35,9 @@ let GridContextMenuComponent = class GridContextMenuComponent {
             if (e.key === 'Escape')
                 this.close.emit();
         };
-        effect(() => {
-            // Re-register listeners when component renders
-            document.addEventListener('mousedown', this.clickOutsideHandler, true);
-            document.addEventListener('keydown', this.keyDownHandler, true);
-        });
+        // Register listeners once on init (no signal dependencies needed)
+        document.addEventListener('mousedown', this.clickOutsideHandler, true);
+        document.addEventListener('keydown', this.keyDownHandler, true);
         this.destroyRef.onDestroy(() => {
             document.removeEventListener('mousedown', this.clickOutsideHandler, true);
             document.removeEventListener('keydown', this.keyDownHandler, true);

package/dist/esm/services/datagrid-state.service.js

@@ -108,9 +108,14 @@ let DataGridStateService = class DataGridStateService {
             const order = p.columnOrder;
             if (!order?.length)
                 return filtered;
+            // Build index map for O(1) lookup instead of repeated O(n) indexOf
+            const orderMap = new Map();
+            for (let i = 0; i < order.length; i++) {
+                orderMap.set(order[i], i);
+            }
             return [...filtered].sort((a, b) => {
-                const ia = order.indexOf(a.columnId);
-                const ib = order.indexOf(b.columnId);
+                const ia = orderMap.get(a.columnId) ?? -1;
+                const ib = orderMap.get(b.columnId) ?? -1;
                 if (ia === -1 && ib === -1)
                     return 0;
                 if (ia === -1)
@@ -241,14 +246,20 @@ let DataGridStateService = class DataGridStateService {
             this.resizeObserver.observe(el);
             measure();
         });
-        // Cleanup on destroy
+        // Cleanup on destroy — null out refs to prevent accidental reuse after teardown
         this.destroyRef.onDestroy(() => {
-            if (this.rafId)
+            if (this.rafId) {
                 cancelAnimationFrame(this.rafId);
-            if (this.autoScrollInterval)
+                this.rafId = 0;
+            }
+            if (this.autoScrollInterval) {
                 clearInterval(this.autoScrollInterval);
-            if (this.resizeObserver)
+                this.autoScrollInterval = null;
+            }
+            if (this.resizeObserver) {
                 this.resizeObserver.disconnect();
+                this.resizeObserver = null;
+            }
         });
         // Clean up column sizing overrides for removed columns
         effect(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alaarab/ogrid-angular",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "description": "OGrid Angular – Angular services, signals, and headless components for OGrid data grids.",
   "main": "dist/esm/index.js",
   "module": "dist/esm/index.js",
@@ -22,7 +22,7 @@
   "files": ["dist", "README.md", "LICENSE"],
   "engines": { "node": ">=18" },
   "dependencies": {
-    "@alaarab/ogrid-core": "2.0.4"
+    "@alaarab/ogrid-core": "2.0.5"
   },
   "peerDependencies": {
     "@angular/core": "^21.0.0",