chore: fix all build warnings — clean 0 Warning(s)

- NavigationView: remove redundant GotFocus/LostFocus (inherited from BaseControl)
- LogExporterControl: pragma suppress unused GotFocus/LostFocus (interface requirement)
- Window.Controls.cs: null-safe IsFocused check with pattern match
- WindowEventDispatcher: null-safe IsFocused check with pattern match
- HorizontalGridControl: remove dead _interactiveContentsDirty and _focusFromBackward fields
- ScrollablePanelControl: remove dead _focusFromBackward field

Author: nickprotop

SharpConsoleUI/Controls/HorizontalGridControl/HorizontalGridControl.cs

@@ -80,8 +80,6 @@ public partial class HorizontalGridControl : BaseControl, IInteractiveControl, I
 		/// </summary>
 		public IInteractiveControl? FocusedContent => GetFocusedChildFromCoordinator();
 		private Dictionary<IInteractiveControl, ColumnContainer> _interactiveContents = new Dictionary<IInteractiveControl, ColumnContainer>();
-		private bool _interactiveContentsDirty = true;
-		private bool _focusFromBackward = false;
 		private bool _isEnabled = true;
 		private Dictionary<IInteractiveControl, int> _splitterControls = new Dictionary<IInteractiveControl, int>();
 		private List<SplitterControl> _splitters = new List<SplitterControl>();
@@ -387,7 +385,6 @@ public void AddColumn(ColumnContainer column)
 			lock (_gridLock)
 			{
 				_columns.Add(column);
-				_interactiveContentsDirty = true;
 			}
 
 			// Force DOM rebuild for runtime addition
@@ -431,7 +428,6 @@ public void AddColumn(ColumnContainer column)
 					_splitters.Add(splitter);
 					_splitterControls[splitter] = _columns.Count - 2;
 
-					_interactiveContentsDirty = true;
 				}
 
 				// Subscribe to splitter's move event
@@ -473,7 +469,6 @@ public bool AddSplitter(int leftColumnIndex, SplitterControl splitterControl)
 				_splitters.Add(splitterControl);
 				_splitterControls[splitterControl] = leftColumnIndex;
 
-				_interactiveContentsDirty = true;
 			}
 
 			// Subscribe to splitter's move event
@@ -594,7 +589,6 @@ public void RemoveColumn(ColumnContainer column)
 
 				_splitterControls = updatedSplitters;
 
-				_interactiveContentsDirty = true;
 			}
 
 			// Unsubscribe events outside lock
@@ -622,7 +616,6 @@ public void ClearColumns()
 				_splitters.Clear();
 				_splitterControls.Clear();
 				_savedColumnWidths.Clear();
-				_interactiveContentsDirty = true;
 			}
 
 			foreach (var s in splittersSnapshot)

SharpConsoleUI/Controls/NavigationView/NavigationView.Input.cs

@@ -89,12 +89,6 @@ public bool HasFocus
 		/// <inheritdoc/>
 		public bool CanReceiveFocus => IsEnabled;
 
-		/// <summary>Raised when the control gains focus.</summary>
-		public event EventHandler? GotFocus;
-
-		/// <summary>Raised when the control loses focus.</summary>
-		public event EventHandler? LostFocus;
-
 		/// <inheritdoc/>
 		public bool ProcessKey(ConsoleKeyInfo key)
 		{

SharpConsoleUI/Controls/ScrollablePanelControl/ScrollablePanelControl.cs

@@ -30,7 +30,6 @@ public partial class ScrollablePanelControl : BaseControl, IInteractiveControl,
 		private int _viewportWidth = 0;
 		private bool _isEnabled = true;
 		private IInteractiveControl? _lastInternalFocusedChild = null;
-		private bool _focusFromBackward = false;
 		// When true, GetInitialFocus returns 'this' so SetFocus(panel) enters scroll mode directly.
 		private bool _enterScrollModeOnNextInitialFocus = false;
 

SharpConsoleUI/Plugins/DeveloperTools/LogExporterControl.cs

@@ -162,12 +162,13 @@ public int? Width
 	/// <inheritdoc/>
 	public bool CanReceiveFocus => _enabled && _visible;
 
+	#pragma warning disable CS0067 // Events required by IFocusableControl but not raised by this legacy control
 	/// <inheritdoc/>
 	public event EventHandler? GotFocus;
 
 	/// <inheritdoc/>
 	public event EventHandler? LostFocus;
-
+	#pragma warning restore CS0067
 
 	/// <inheritdoc/>
 	public bool WantsMouseEvents => true;

SharpConsoleUI/Window.Controls.cs

@@ -80,7 +80,7 @@ public void RemoveContent(IWindowControl content)
 			// Handle focus logic before removing
 			if (content is IInteractiveControl interactiveControl)
 			{
-				bool wasFocused = FocusManager.IsFocused(interactiveControl as IFocusableControl);
+				bool wasFocused = interactiveControl is IFocusableControl fc && FocusManager.IsFocused(fc);
 
 				if (wasFocused)
 				{

SharpConsoleUI/Windows/WindowEventDispatcher.cs

@@ -639,7 +639,7 @@ public bool HasActiveInteractiveContent(out IInteractiveControl? interactiveCont
 
 			// Legacy fallback during migration — keep existing logic here unchanged
 			// First try to find focused control in _interactiveContents (direct children)
-			interactiveContent = _window._interactiveContents.LastOrDefault(ic => ic.IsEnabled && _window.FocusManager.IsFocused(ic as Controls.IFocusableControl));
+			interactiveContent = _window._interactiveContents.LastOrDefault(ic => ic.IsEnabled && ic is Controls.IFocusableControl fc && _window.FocusManager.IsFocused(fc));
 
 			// Fallback: route keyboard to the control that currently has focus via FocusManager.
 			// This handles nested controls in containers (e.g. a leaf inside ScrollablePanel).