From 25ff61fe4735db5dab51b0f398a832d754dbe45f Mon Sep 17 00:00:00 2001
From: Tsanie Lily <tsorgy@gmail.com>
Date: Wed, 26 Apr 2023 10:02:21 +0800
Subject: [PATCH] optimize

---
 lib/ui/grid/grid.js | 69 +++------------------------------------------
 1 file changed, 4 insertions(+), 65 deletions(-)

diff --git a/lib/ui/grid/grid.js b/lib/ui/grid/grid.js
index 11f0e84..d1c9fe0 100644
--- a/lib/ui/grid/grid.js
+++ b/lib/ui/grid/grid.js
@@ -223,7 +223,7 @@ class Grid {
         this.#parent = container;
         const grid = createElement('div', 'ui-grid');
         grid.setAttribute('tabindex', 0);
-        grid.addEventListener('keydown', e => {
+        const onKeydown = e => {
             let index = this.selectedIndex;
             let flag = false;
             if (e.key === 'ArrowUp') {
@@ -249,7 +249,8 @@ class Grid {
                 }
                 e.stopPropagation();
             }
-        });
+        };
+        grid.addEventListener('keydown', e => throttle(onKeydown, 800, this, e));
         container.replaceChildren(grid);
         const sizer = createElement('span', 'ui-grid-sizer');
         grid.appendChild(sizer);
@@ -581,68 +582,6 @@ class Grid {
         return body;
     }
 
-    #adjustRows() {
-        let count = this.#rowCount;
-        if (isNaN(count) || count < 0 || !this.virtual) {
-            count = this.#currentSource.length;
-        }
-        const cols = this.columns;
-        const content = this.#refs.bodyContent;
-        const exists = content.children.length;
-        count -= exists;
-        if (count > 0) {
-            for (let i = 0; i < count; i += 1) {
-                const row = createElement('tr', 'ui-grid-row');
-                // row.addEventListener('mousedown', e => this.#onRowClicked(e, exists + i));
-                // row.addEventListener('dblclick', e => this.#onRowDblClicked(e));
-                cols.forEach((col, j) => {
-                    const cell = createElement('td');
-                    if (col.visible !== false) {
-                        cell.dataset.row = String(exists + i);
-                        cell.dataset.col = String(j);
-                        const style = this.#get(col.key, 'style');
-                        if (style != null) {
-                            for (let css of Object.entries(style)) {
-                                cell.style.setProperty(css[0], css[1]);
-                            }
-                        }
-                        if (col.css != null) {
-                            for (let css of Object.entries(col.css)) {
-                                cell.style.setProperty(css[0], css[1]);
-                            }
-                        }
-                        if (Grid.ColumnTypes.isCheckbox(col.type)) {
-                            cell.appendChild(GridCheckboxColumn.createEdit(e => this.#onRowChanged(e, exists + i, col, e.target.checked)));
-                            // this.#colTypes[col.key] = GridCheckboxColumn;
-                        } else {
-                            let type = this.#colTypes[col.key];
-                            if (type == null) {
-                                if (isNaN(col.type)) {
-                                    if (this.allowHtml && col.type != null) {
-                                        type = col.type;
-                                    }
-                                } else {
-                                    type = ColumnTypes[col.type];
-                                }
-                                type ??= GridColumn;
-                                this.#colTypes[col.key] = type;
-                            }
-                            cell.appendChild(type.create(col));
-                        }
-                    }
-                    row.appendChild(cell);
-                });
-                row.appendChild(createElement('td'));
-                content.appendChild(row);
-            }
-        } else if (count < 0) {
-            for (let i = -1; i >= count; i -= 1) {
-                // content.removeChild(content.children[exists + i]);
-                content.children[exists + i].remove();
-            }
-        }
-    }
-
     #fillRows(widths) {
         let count = this.#rowCount;
         if (isNaN(count) || count < 0 || !this.virtual) {
@@ -737,7 +676,7 @@ class Grid {
             if (!isPositive(row.children.length)) {
                 return;
             }
-            row.dataset.row = String(index);
+            // row.dataset.row = String(index);
             const item = vals.values;
             const selected = selectedIndexes.indexOf(index) >= 0;
             if (selected) {