More Toplevel.cs organization & docs

Terminal.Gui/Application/Application.Keyboard.cs

@@ -230,9 +230,14 @@ public static partial class Application // Keyboard handling
     ///     This is an internal method used by the <see cref="View"/> class to add Application key bindings.
     /// </remarks>
     /// <param name="key">The key being bound.</param>
-    /// <param name="view">The view that is bound to the key.</param>
-    internal static void AddKeyBinding (Key key, View view)
+    /// <param name="view">The view that is bound to the key. If <see langword="null"/>, <see cref="Application.Current"/> will be used.</param>
+    internal static void AddKeyBinding (Key key, [CanBeNull] View view)
     {
+        if (view is null)
+        {
+            view = Current;
+        }
+
         if (!_keyBindings.ContainsKey (key))
         {
             _keyBindings [key] = [];

Terminal.Gui/Application/Application.Run.cs

@@ -577,6 +577,7 @@ public static partial class Application // Run (Begin, Run, End, Stop)
             Iteration?.Invoke (null, new ());
             EnsureModalOrVisibleAlwaysOnTop (state.Toplevel);
 
+            // TODO: Overlapped - Move elsewhere
             if (state.Toplevel != Current)
             {
                 OverlappedTop?.OnDeactivate (state.Toplevel);

Terminal.Gui/Application/Application.Toplevel.cs

@@ -18,7 +18,7 @@ public static partial class Application // Toplevel handling
     ///     <see cref="Toplevel"/> .
     /// </summary>
     /// <remarks>
-    ///     Only relevant in scenarios where <see cref="Toplevel.IsOverlappedContainer"/> is <see langword="true"/>.
+    ///     This will only be distinct from <see cref="Application.Top"/> in scenarios where <see cref="Toplevel.IsOverlappedContainer"/> is <see langword="true"/>.
     /// </remarks>
     /// <value>The current.</value>
     public static Toplevel Current { get; private set; }