Window -> not sizeable/movable by default.

Window: Code cleanup.
Frameview: Code cleanup.

Author: tig

Terminal.Gui/Views/FrameView.cs

@@ -1,9 +1,21 @@
-namespace Terminal.Gui;
+#nullable enable
+namespace Terminal.Gui;
 
 // TODO: FrameView is mis-named, really. It's far more about it being a TabGroup than a frame. 
+
 /// <summary>
-///     The FrameView is a container View with a border around it. 
+///     A non-overlapped container for other views with a border and optional title.
 /// </summary>
+/// <remarks>
+///     <para>
+///         FrameView has <see cref="View.BorderStyle"/> set to <see cref="LineStyle.Single"/> and
+///         inherits it's color scheme from the <see cref="SuperView"/>.
+///     </para>
+///     <para>
+///         
+///     </para>
+/// </remarks>
+/// <seealso cref="Window"/>
 public class FrameView : View
 {
     /// <summary>
@@ -14,21 +26,9 @@ public FrameView ()
     {
         CanFocus = true;
         TabStop = TabBehavior.TabGroup;
-        Border.Thickness = new Thickness (1);
-        Border.LineStyle = DefaultBorderStyle;
-
-        //Border.ColorScheme = ColorScheme;
-        Border.Data = "Border";
-        MouseClick += FrameView_MouseClick;
-    }
-
-    private void FrameView_MouseClick (object sender, MouseEventArgs e)
-    {
-        // base sets focus on HotKey
-        e.Handled = InvokeCommand<KeyBinding> (Command.HotKey, new ([Command.HotKey], this, this)) == true;
+        BorderStyle = DefaultBorderStyle;
     }
 
-
     /// <summary>
     ///     The default <see cref="LineStyle"/> for <see cref="FrameView"/>'s border. The default is
     ///     <see cref="LineStyle.Single"/>.

Terminal.Gui/Views/Window.cs

@@ -1,38 +1,41 @@
-namespace Terminal.Gui;
+#nullable enable
+namespace Terminal.Gui;
 
 /// <summary>
-///     A <see cref="Toplevel"/> <see cref="View"/> with <see cref="View.BorderStyle"/> set to
-///     <see cref="LineStyle.Single"/>. Provides a container for other views.
+///     An overlapped container for other views with a border and optional title.
 /// </summary>
 /// <remarks>
 ///     <para>
-///         If any subview is a button and the <see cref="Button.IsDefault"/> property is set to true, the Enter key will
-///         invoke the <see cref="Command.Accept"/> command on that subview.
+///         Window has <see cref="View.BorderStyle"/> set to <see cref="LineStyle.Single"/>, <see cref="View.Arrangement"/>
+///         set to <see cref="ViewArrangement.Overlapped"/>, and
+///         uses the Base <see cref="Colors.ColorSchemes"/> color scheme by default.
+///     </para>
+///     <para>
+///         To enable Window to be sized and moved by the user, adjust <see cref="View.Arrangement"/>.
 ///     </para>
 /// </remarks>
+/// <seealso cref="FrameView"/>
 public class Window : Toplevel
 {
-
-    /// <summary>
-    /// Gets or sets whether all <see cref="Window"/>s are shown with a shadow effect by default.
-    /// </summary>
-    [SerializableConfigurationProperty (Scope = typeof (ThemeScope))]
-    public static ShadowStyle DefaultShadow { get; set; } = ShadowStyle.None;
-
-
     /// <summary>
     ///     Initializes a new instance of the <see cref="Window"/> class.
     /// </summary>
     public Window ()
     {
         CanFocus = true;
         TabStop = TabBehavior.TabGroup;
-        Arrangement = ViewArrangement.Movable | ViewArrangement.Overlapped | ViewArrangement.Resizable;
-        ColorScheme = Colors.ColorSchemes ["Base"]; // TODO: make this a theme property
+        Arrangement = ViewArrangement.Overlapped;
+        base.ColorScheme = Colors.ColorSchemes ["Base"]; // TODO: make this a theme property
         BorderStyle = DefaultBorderStyle;
-        ShadowStyle = DefaultShadow;
+        base.ShadowStyle = DefaultShadow;
     }
 
+    /// <summary>
+    ///     Gets or sets whether all <see cref="Window"/>s are shown with a shadow effect by default.
+    /// </summary>
+    [SerializableConfigurationProperty (Scope = typeof (ThemeScope))]
+    public static ShadowStyle DefaultShadow { get; set; } = ShadowStyle.None;
+
     // TODO: enable this
     ///// <summary>
     ///// The default <see cref="LineStyle"/> for <see cref="Window"/>'s border. The default is <see cref="LineStyle.Single"/>.

UICatalog/Scenarios/Adornments.cs

@@ -34,7 +34,7 @@ public override void Main ()
         var window = new Window
         {
             Title = "The _Window",
-            Arrangement = ViewArrangement.Movable,
+            Arrangement = ViewArrangement.Overlapped | ViewArrangement.Movable,
 
             Width = Dim.Fill (Dim.Func (() => editor.Frame.Width )),
             Height = Dim.Fill ()