Fix code review feedback: improve editing UX and accessibility

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/components/src/renderers/complex/data-table.tsx

@@ -355,9 +355,12 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
     setEditValue(currentValue ?? '');
   };
 
-  const saveEdit = () => {
+  const saveEdit = (force: boolean = false) => {
     if (!editingCell) return;
 
+    // Don't save if we're in cancelled state (unless forced)
+    if (!force && editingCell === null) return;
+    
     const { rowIndex, columnKey } = editingCell;
     const globalIndex = (currentPage - 1) * pageSize + rowIndex;
     const row = sortedData[globalIndex];
@@ -379,6 +382,9 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
   const handleCellKeyDown = (e: React.KeyboardEvent, rowIndex: number, columnKey: string) => {
     if (!editable) return;
 
+    const column = columns.find(col => col.accessorKey === columnKey);
+    if (column?.editable === false) return;
+    
     if (e.key === 'Enter' && !editingCell) {
       e.preventDefault();
       const value = paginatedData[rowIndex][columnKey];
@@ -389,7 +395,7 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
   const handleEditKeyDown = (e: React.KeyboardEvent) => {
     if (e.key === 'Enter') {
       e.preventDefault();
-      saveEdit();
+      saveEdit(true);
     } else if (e.key === 'Escape') {
       e.preventDefault();
       cancelEdit();
@@ -596,15 +602,14 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
                             }}
                             onDoubleClick={() => isEditable && startEdit(rowIndex, col.accessorKey, cellValue)}
                             onKeyDown={(e) => handleCellKeyDown(e, rowIndex, col.accessorKey)}
-                            tabIndex={isEditable ? 0 : -1}
+                            tabIndex={0}
                           >
                             {isEditing ? (
                               <Input
                                 ref={editInputRef}
                                 value={editValue}
                                 onChange={(e) => setEditValue(e.target.value)}
                                 onKeyDown={handleEditKeyDown}
-                                onBlur={saveEdit}
                                 className="h-8 px-2 py-1"
                               />
                             ) : (