Fixed ProgressBar to not set Height

Terminal.Gui/Views/ProgressBar.cs

@@ -61,7 +61,7 @@ public class ProgressBar : View
         set
         {
             _bidirectionalMarquee = value;
-            SetNeedsDisplay ();
+            ContentSize = Viewport.Size with { Height = 1 };
         }
     }
 
@@ -74,7 +74,7 @@ public class ProgressBar : View
         {
             _fraction = Math.Min (value, 1);
             _isActivity = false;
-            SetNeedsDisplay ();
+            ContentSize = Viewport.Size with { Height = 1 };
         }
     }
 
@@ -109,7 +109,7 @@ public class ProgressBar : View
                     break;
             }
 
-            SetNeedsDisplay ();
+            ContentSize = Viewport.Size with { Height = 1 };
         }
     }
 
@@ -120,7 +120,7 @@ public class ProgressBar : View
         set
         {
             _segmentCharacter = value;
-            SetNeedsDisplay ();
+            ContentSize = Viewport.Size with { Height = 1 };
         }
     }
 
@@ -181,7 +181,7 @@ public class ProgressBar : View
 
         if (ProgressBarFormat != ProgressBarFormat.Simple && !_isActivity)
         {
-            var tf = new TextFormatter { Alignment = TextAlignment.Centered, Text = Text };
+            var tf = new TextFormatter { Alignment = TextAlignment.Centered, Text = Text, AutoSize = true};
             var attr = new Attribute (ColorScheme.HotNormal.Foreground, ColorScheme.HotNormal.Background);
 
             if (_fraction > .5)
@@ -269,24 +269,19 @@ public class ProgressBar : View
 
     private void ProgressBar_Initialized (object sender, EventArgs e)
     {
+        ContentSize = new (0, 1);
+
         ColorScheme = new ColorScheme (ColorScheme ?? SuperView?.ColorScheme ?? Colors.ColorSchemes ["Base"])
         {
             HotNormal = new Attribute (Color.BrightGreen, Color.Gray)
         };
     }
 
-    private void ProgressBar_LayoutStarted (object sender, EventArgs e)
-    {
-        // TODO: use Dim.Auto
-        Height = 1 + GetAdornmentsThickness ().Vertical;
-    }
-
     private void SetInitialProperties ()
     {
-        Height = 1; // This will be updated when Viewport is updated in ProgressBar_LayoutStarted
+        Height = Dim.Auto (Dim.DimAutoStyle.Content);
         CanFocus = false;
         _fraction = 0;
-        LayoutStarted += ProgressBar_LayoutStarted;
         Initialized += ProgressBar_Initialized;
     }
 }

UnitTests/View/Layout/Dim.AutoTests.cs

@@ -1191,7 +1191,7 @@ public class DimAutoTests (ITestOutputHelper output)
 
 
     [Theory]
-    [InlineData("01234", 5, 5)]
+    [InlineData ("01234", 5, 5)]
     [InlineData ("01234", 6, 6)]
     [InlineData ("01234", 4, 5)]
     [InlineData ("01234", 0, 5)]
@@ -1216,5 +1216,17 @@ public class DimAutoTests (ITestOutputHelper output)
 
     }
 
+    [Fact]
+    public void DimAutoStyle_Content_UsesContentSize_If_No_Subviews ()
+    {
+        DimAutoTestView view = new (Auto (DimAutoStyle.Content), Auto (DimAutoStyle.Content));
+        view.ContentSize = new (5, 5);
+        view.SetRelativeLayout (new (10, 10));
+
+        Assert.Equal (new (5, 5), view.Frame.Size);
+
+
+    }
+
     // Test variations of Frame
 }