From 6d0ac9a5417bdc01048b5e86f9e4a985f58ca091 Mon Sep 17 00:00:00 2001
From: Dariusz Komosinski <darek.komosinski@gmail.com>
Date: Thu, 13 May 2021 16:33:49 +0200
Subject: [PATCH] Fix property type on Stretch.

---
 samples/ControlCatalog/Pages/ViewboxPage.xaml    |  7 ++++---
 samples/ControlCatalog/Pages/ViewboxPage.xaml.cs | 16 +++-------------
 src/Avalonia.Controls/Viewbox.cs                 |  3 ++-
 3 files changed, 9 insertions(+), 17 deletions(-)

diff --git a/samples/ControlCatalog/Pages/ViewboxPage.xaml b/samples/ControlCatalog/Pages/ViewboxPage.xaml
index 6e649b39a5..ef802db33e 100644
--- a/samples/ControlCatalog/Pages/ViewboxPage.xaml
+++ b/samples/ControlCatalog/Pages/ViewboxPage.xaml
@@ -12,8 +12,9 @@
 
           <Border HorizontalAlignment="Center" Grid.Column="0" BorderThickness="1" BorderBrush="Orange" Width="200" Height="200">
             <Border VerticalAlignment="Center" HorizontalAlignment="Center" Grid.Column="0" BorderThickness="1" BorderBrush="CornflowerBlue" Width="{Binding #WidthSlider.Value}" Height="{Binding #HeightSlider.Value}" >
-              <Viewbox x:Name="Viewbox"
-                StretchDirection="{Binding #StretchDirectionSelector.SelectedItem}">
+              <Viewbox
+                  Stretch="{Binding #StretchSelector.SelectedItem}"
+                  StretchDirection="{Binding #StretchDirectionSelector.SelectedItem}">
                 <Ellipse Width="50" Height="50" Fill="CornflowerBlue" />
               </Viewbox>
             </Border>
@@ -25,7 +26,7 @@
             <TextBlock Text="Height" />
             <Slider Minimum="10" Maximum="200" Value="100" x:Name="HeightSlider" TickFrequency="25" TickPlacement="TopLeft" />
             <TextBlock Text="Stretch" />
-            <ComboBox x:Name="StretchSelector" HorizontalAlignment="Stretch" Margin="0,0,0,2" SelectionChanged="StretchSelector_OnSelectionChanged" />
+            <ComboBox x:Name="StretchSelector" HorizontalAlignment="Stretch" Margin="0,0,0,2" />
             <TextBlock Text="Stretch Direction" />
             <ComboBox x:Name="StretchDirectionSelector" HorizontalAlignment="Stretch" />
           </StackPanel>
diff --git a/samples/ControlCatalog/Pages/ViewboxPage.xaml.cs b/samples/ControlCatalog/Pages/ViewboxPage.xaml.cs
index 7a2bceb8f2..94b3f3ea14 100644
--- a/samples/ControlCatalog/Pages/ViewboxPage.xaml.cs
+++ b/samples/ControlCatalog/Pages/ViewboxPage.xaml.cs
@@ -6,23 +6,18 @@ namespace ControlCatalog.Pages
 {
     public class ViewboxPage : UserControl
     {
-        private readonly Viewbox _viewbox;
-        private readonly ComboBox _stretchSelector;
-
         public ViewboxPage()
         {
             InitializeComponent();
 
-            _viewbox = this.FindControl<Viewbox>("Viewbox");
-
-            _stretchSelector = this.FindControl<ComboBox>("StretchSelector");
+            var stretchSelector = this.FindControl<ComboBox>("StretchSelector");
 
-            _stretchSelector.Items = new[]
+            stretchSelector.Items = new[]
             {
                 Stretch.Uniform, Stretch.UniformToFill, Stretch.Fill, Stretch.None
             };
 
-            _stretchSelector.SelectedIndex = 0;
+            stretchSelector.SelectedIndex = 0;
 
             var stretchDirectionSelector = this.FindControl<ComboBox>("StretchDirectionSelector");
 
@@ -38,10 +33,5 @@ namespace ControlCatalog.Pages
         {
             AvaloniaXamlLoader.Load(this);
         }
-
-        private void StretchSelector_OnSelectionChanged(object sender, SelectionChangedEventArgs e)
-        {
-            _viewbox.Stretch = (Stretch) _stretchSelector.SelectedItem!;
-        }
     }
 }
diff --git a/src/Avalonia.Controls/Viewbox.cs b/src/Avalonia.Controls/Viewbox.cs
index b65f4b31d8..15ca070de2 100644
--- a/src/Avalonia.Controls/Viewbox.cs
+++ b/src/Avalonia.Controls/Viewbox.cs
@@ -12,7 +12,8 @@ namespace Avalonia.Controls
         /// Defines the <see cref="Stretch"/> property.
         /// </summary>
         public static readonly AvaloniaProperty<Stretch> StretchProperty =
-            AvaloniaProperty.Register<Image, Stretch>(nameof(Stretch), Stretch.Uniform);
+            AvaloniaProperty.RegisterDirect<Viewbox, Stretch>(nameof(Stretch),
+                v => v.Stretch, (c, v) => c.Stretch = v, Stretch.Uniform);
 
         /// <summary>
         /// Defines the <see cref="StretchDirection"/> property.