Prevent editing error where selection can end up temporarily set to o…

…ut of bounds locations
glideapps · Apr 10, 2022 · 0e62b1c · 0e62b1c
1 parent 9f13f2e
commit 0e62b1c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/core/src/data-editor/data-editor.tsx b/packages/core/src/data-editor/data-editor.tsx
@@ -1695,15 +1695,15 @@ const DataEditorImpl: React.ForwardRefRenderFunction<DataEditorRef, DataEditorPr
                 const isEditingTrailingRow =
                     gridSelection.current.cell[1] === mangledRows - 1 && newValue !== undefined;
                 updateSelectedCell(
-                    gridSelection.current.cell[0] + movX,
-                    gridSelection.current.cell[1] + movY,
+                    clamp(gridSelection.current.cell[0] + movX, 0, mangledCols.length - 1),
+                    clamp(gridSelection.current.cell[1] + movY, 0, mangledRows - 1),
                     isEditingTrailingRow,
                     false
                 );
             }
             onFinishedEditing?.(newValue, movement);
         },
-        [overlay?.cell, focus, gridSelection, onFinishedEditing, mangledOnCellEdited, mangledRows, updateSelectedCell]
+        [overlay?.cell, focus, gridSelection, onFinishedEditing, mangledOnCellEdited, mangledRows, updateSelectedCell, mangledCols.length]
     );
 
     const overlayID = React.useMemo(() => {