ProgressBar MinWidth & MinHeight set in template

src/Avalonia.Controls/ProgressBar.cs

@@ -26,12 +26,13 @@ namespace Avalonia.Controls
 
         static ProgressBar()
         {
+            PseudoClass(OrientationProperty, o => o == Avalonia.Controls.Orientation.Vertical, ":vertical");
+            PseudoClass(OrientationProperty, o => o == Avalonia.Controls.Orientation.Horizontal, ":horizontal");
+
             ValueProperty.Changed.AddClassHandler<ProgressBar>(x => x.ValueChanged);
 
             IsIndeterminateProperty.Changed.AddClassHandler<ProgressBar>(
                 (p, e) => { if (p._indicator != null) p.UpdateIsIndeterminate((bool)e.NewValue); });
-            OrientationProperty.Changed.AddClassHandler<ProgressBar>(
-                (p, e) => { if (p._indicator != null) p.UpdateOrientation((Orientation)e.NewValue); });
         }
 
         public bool IsIndeterminate
@@ -59,7 +60,6 @@ namespace Avalonia.Controls
             _indicator = e.NameScope.Get<Border>("PART_Indicator");
 
             UpdateIndicator(Bounds.Size);
-            UpdateOrientation(Orientation);
             UpdateIsIndeterminate(IsIndeterminate);
         }
 
@@ -86,26 +86,6 @@ namespace Avalonia.Controls
             }
         }
 
-        private void UpdateOrientation(Orientation orientation)
-        {
-            if (orientation == Orientation.Horizontal)
-            {
-                MinHeight = Math.Min(14, double.IsNaN(Height) ? double.MaxValue : Height);
-                MinWidth = Math.Min(200, double.IsNaN(Width) ? double.MaxValue : Width);
-
-                _indicator.HorizontalAlignment = HorizontalAlignment.Left;
-                _indicator.VerticalAlignment = VerticalAlignment.Stretch;
-            }
-            else
-            {
-                MinHeight = Math.Min(200, double.IsNaN(Height) ? double.MaxValue : Height);
-                MinWidth = Math.Min(14, double.IsNaN(Width) ? double.MaxValue : Width);
-
-                _indicator.HorizontalAlignment = HorizontalAlignment.Stretch;
-                _indicator.VerticalAlignment = VerticalAlignment.Bottom;
-            }
-        }
-
         private void UpdateIsIndeterminate(bool isIndeterminate)
         {
             if (isIndeterminate)

src/Avalonia.Themes.Default/ProgressBar.xaml

@@ -1,20 +1,38 @@
-<Style xmlns="https://github.com/avaloniaui" Selector="ProgressBar">
+<Styles xmlns="https://github.com/avaloniaui">
-  <Setter Property="Background" Value="{DynamicResource ThemeAccentBrush4}"/>
+  <Style Selector="ProgressBar">
-  <Setter Property="Foreground" Value="{DynamicResource ThemeAccentBrush}"/>
+    <Setter Property="Background" Value="{DynamicResource ThemeAccentBrush4}"/>
-  <Setter Property="Template">
+    <Setter Property="Foreground" Value="{DynamicResource ThemeAccentBrush}"/> 
-    <ControlTemplate>
+    <Setter Property="Template">
-      <Border Background="{TemplateBinding Background}"
+      <ControlTemplate>
-              BorderBrush="{TemplateBinding BorderBrush}"
+        <Border Background="{TemplateBinding Background}"
-              BorderThickness="{TemplateBinding BorderThickness}">
+                BorderBrush="{TemplateBinding BorderBrush}"
-        <Grid>
+                BorderThickness="{TemplateBinding BorderThickness}">
-          <Border Name="PART_Track"
+          <Grid>
-                  BorderThickness="1"
+            <Border Name="PART_Track"
-                  BorderBrush="{TemplateBinding Background}"/>
+                    BorderThickness="1"
-          <Border Name="PART_Indicator"
+                    BorderBrush="{TemplateBinding Background}"/>
-                  BorderThickness="1"
+            <Border Name="PART_Indicator"
-                  Background="{TemplateBinding Foreground}"/>
+                    BorderThickness="1"
-        </Grid>
+                    Background="{TemplateBinding Foreground}" />
-      </Border>
+          </Grid>
-    </ControlTemplate>
+        </Border>
-  </Setter>
+      </ControlTemplate>
-</Style>
+    </Setter>
+  </Style>
+  <Style Selector="ProgressBar:horizontal /template/ Border#PART_Indicator">
+    <Setter Property="HorizontalAlignment" Value="Left"/>
+    <Setter Property="VerticalAlignment" Value="Stretch"/>
+  </Style>
+  <Style Selector="ProgressBar:vertical /template/ Border#PART_Indicator">
+    <Setter Property="HorizontalAlignment" Value="Stretch"/>
+    <Setter Property="VerticalAlignment" Value="Bottom"/>
+  </Style>
+  <Style Selector="ProgressBar:horizontal">
+    <Setter Property="MinWidth" Value="200"/>
+    <Setter Property="MinHeight" Value="14"/>
+  </Style>
+  <Style Selector="ProgressBar:vertical">
+    <Setter Property="MinWidth" Value="14"/>
+    <Setter Property="MinHeight" Value="200"/>
+  </Style>
+</Styles>