fix(table): checkbox mode sync with selection, click-on-checkbox toggle

Extract ToggleRowSelection for keyboard spacebar. Add checkbox column
click detection so clicking the checkbox toggles without clearing other
selections. Sync IsChecked state with selection in SelectAll, ClearSelection,
ToggleRowSelection, and SelectRange.

Author: nickprotop

SharpConsoleUI/Controls/TableControl/TableControl.Keyboard.cs

@@ -191,19 +191,8 @@ public bool ProcessKey(ConsoleKeyInfo key)
 			case ConsoleKey.Spacebar when _checkboxMode && _multiSelectEnabled:
 				if (_selectedRowIndex >= 0)
 				{
-					int dataIdx = MapDisplayToData(_selectedRowIndex);
-					if (_dataSource == null)
-					{
-						lock (_tableLock)
-						{
-							if (dataIdx >= 0 && dataIdx < _rows.Count)
-							{
-								_rows[dataIdx].IsChecked = !_rows[dataIdx].IsChecked;
-								Container?.Invalidate(true);
-								return true;
-							}
-						}
-					}
+					ToggleRowSelection(_selectedRowIndex);
+					return true;
 				}
 				return false;
 

SharpConsoleUI/Controls/TableControl/TableControl.Mouse.cs

@@ -203,7 +203,7 @@ public bool ProcessMouseEvent(MouseEventArgs args)
 			int rowIdx = GetRowIndexAtY(args.Position.Y);
 			if (rowIdx >= 0)
 			{
-				// Multi-select: Ctrl+Click toggles, Shift+Click extends range
+				// Multi-select: Ctrl+Click toggles, Shift+Click extends range, checkbox click toggles
 				if (_multiSelectEnabled && args.HasFlag(MouseFlags.ButtonCtrl))
 				{
 					ToggleRowSelection(rowIdx);
@@ -216,10 +216,27 @@ public bool ProcessMouseEvent(MouseEventArgs args)
 					SelectRange(anchor, rowIdx);
 					SetSelectedRow(rowIdx);
 				}
+				else if (_checkboxMode && IsClickOnCheckbox(args.Position.X))
+				{
+					// Click on checkbox column — toggle without clearing other selections
+					ToggleRowSelection(rowIdx);
+					SetSelectedRow(rowIdx);
+				}
 				else
 				{
 					if (_multiSelectEnabled)
+					{
+						// Clear checkboxes when clicking outside checkbox column
+						if (_checkboxMode && _dataSource == null)
+						{
+							lock (_tableLock)
+							{
+								foreach (var row in _rows)
+									row.IsChecked = false;
+							}
+						}
 						_selectedRowIndices.Clear();
+					}
 					SetSelectedRow(rowIdx);
 				}
 
@@ -345,6 +362,14 @@ private int GetColumnIndexAtX(int relativeX)
 		return -1;
 	}
 
+	private bool IsClickOnCheckbox(int relativeX)
+	{
+		if (!_checkboxMode) return false;
+		// Checkbox is rendered as "[x] " or "[ ] " — 4 chars at the start of the row
+		int checkboxEnd = Margin.Left + (_borderStyle != BorderStyle.None ? 1 : 0) + 4;
+		return relativeX < checkboxEnd;
+	}
+
 	private bool IsClickOnHeader(MouseEventArgs args)
 	{
 		if (!_showHeader) return false;

SharpConsoleUI/Controls/TableControl/TableControl.Selection.cs

@@ -219,7 +219,11 @@ public void SelectAll()
 		int count = RowCount;
 		_selectedRowIndices.Clear();
 		for (int i = 0; i < count; i++)
+		{
 			_selectedRowIndices.Add(i);
+			if (_checkboxMode)
+				SyncCheckboxState(i);
+		}
 		Container?.Invalidate(true);
 	}
 
@@ -228,6 +232,14 @@ public void SelectAll()
 	/// </summary>
 	public void ClearSelection()
 	{
+		if (_checkboxMode && _dataSource == null)
+		{
+			lock (_tableLock)
+			{
+				foreach (var row in _rows)
+					row.IsChecked = false;
+			}
+		}
 		_selectedRowIndices.Clear();
 		Container?.Invalidate(true);
 	}
@@ -272,6 +284,11 @@ internal void ToggleRowSelection(int displayIndex)
 			_selectedRowIndices.Remove(displayIndex);
 		else
 			_selectedRowIndices.Add(displayIndex);
+
+		// Sync checkbox state
+		if (_checkboxMode)
+			SyncCheckboxState(displayIndex);
+
 		Container?.Invalidate(true);
 	}
 
@@ -284,10 +301,28 @@ internal void SelectRange(int fromDisplayIndex, int toDisplayIndex)
 		int start = Math.Min(fromDisplayIndex, toDisplayIndex);
 		int end = Math.Max(fromDisplayIndex, toDisplayIndex);
 		for (int i = start; i <= end; i++)
+		{
 			_selectedRowIndices.Add(i);
+			if (_checkboxMode)
+				SyncCheckboxState(i);
+		}
 		Container?.Invalidate(true);
 	}
 
+	/// <summary>
+	/// Syncs the IsChecked property of a row with its selection state.
+	/// </summary>
+	private void SyncCheckboxState(int displayIndex)
+	{
+		if (_dataSource != null) return;
+		lock (_tableLock)
+		{
+			int dataIdx = MapDisplayToData(displayIndex);
+			if (dataIdx >= 0 && dataIdx < _rows.Count)
+				_rows[dataIdx].IsChecked = _selectedRowIndices.Contains(displayIndex);
+		}
+	}
+
 	/// <summary>
 	/// Returns whether a display row index is selected (in multi-select mode) or is the current selected row.
 	/// </summary>