From 00a2ed29b117ef9fb1740b68c0b5b869f995b1a6 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:09:06 +0100
Subject: [PATCH 1/9] Added ScrollViewer for peg list

---
 src/Codebreaker.WPF/Views/Pages/GamePage.xaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
index b2852d29..5f503032 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
@@ -94,13 +94,13 @@
             Margin="86,0,0,8"
             ViewModel="{Binding ViewModel, ElementName=Self, Mode=OneWay}" />
 
+        <ScrollViewer Grid.Row="2" Padding="0,0,0,15" x:Name="pegScrollViewer">
         <ListView
             Height="Auto"
             Visibility="{Binding ViewModel.GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibilityConverter}, ConverterParameter=Running}"
-            Grid.Row="2"
-            Background="LightGray"
             ItemsSource="{Binding ViewModel.GameMoves, Mode=OneWay}"
             ItemTemplate="{StaticResource PegsTemplate}" />
+        </ScrollViewer>
 
         <TextBlock Grid.Row="3" Text="{Binding ViewModel.GameStatus,Mode=OneWay}" />
             <ProgressBar

From 232490502f155ff928f2dad9fde032fd123418ba Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:09:28 +0100
Subject: [PATCH 2/9] Made ScrollViewer automatically scroll to last move

---
 src/Codebreaker.WPF/Helpers/PageExtensions.cs | 31 +++++++++++++++++++
 .../Views/Pages/GamePage.xaml.cs              | 22 ++++++++-----
 2 files changed, 45 insertions(+), 8 deletions(-)
 create mode 100644 src/Codebreaker.WPF/Helpers/PageExtensions.cs

diff --git a/src/Codebreaker.WPF/Helpers/PageExtensions.cs b/src/Codebreaker.WPF/Helpers/PageExtensions.cs
new file mode 100644
index 00000000..3e4361cf
--- /dev/null
+++ b/src/Codebreaker.WPF/Helpers/PageExtensions.cs
@@ -0,0 +1,31 @@
+using CommunityToolkit.Mvvm.Messaging;
+
+namespace Codebreaker.WPF.Helpers;
+
+internal static class PageExtensions
+{
+    public static void UnregisterAllOnUnloaded(this IMessenger messenger, FrameworkElement page) =>
+        messenger.UnregisterAllOnUnloaded(page, page);
+
+    public static void UnregisterAllOnUnloaded(this IMessenger messenger, FrameworkElement page, object messageRecepient)
+    {
+        void UnloadedCallback(object sender, RoutedEventArgs args)
+        {
+            messenger.UnregisterAll(messageRecepient);
+            page.Unloaded -= UnloadedCallback;
+        }
+
+        page.Unloaded += UnloadedCallback;
+    }
+
+    public static void CallOnceOnUnloaded(this FrameworkElement page, Action<object, RoutedEventArgs> action)
+    {
+        void Callback(object sender, RoutedEventArgs args)
+        {
+            action?.Invoke(sender, args);
+            page.Unloaded -= Callback;
+        }
+
+        page.Unloaded += Callback;
+    }
+}
diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml.cs b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml.cs
index 23650aaf..db85fa13 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml.cs
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml.cs
@@ -1,8 +1,10 @@
 using Codebreaker.ViewModels.Contracts.Services;
+using Codebreaker.WPF.Helpers;
+using CommunityToolkit.Mvvm.Messaging;
 
 namespace Codebreaker.WPF.Views.Pages;
 
-public partial class GamePage : Page
+public partial class GamePage : Page, IRecipient<GameMoveMessage>
 {
     private readonly INavigationService _navigationService;
 
@@ -10,15 +12,10 @@ public GamePage()
     {
         ViewModel = App.Current.GetService<GamePageViewModel>();
         _navigationService = App.Current.GetService<INavigationService>();
-
         DataContext = this;
-
         InitializeComponent();
-
-        //WeakReferenceMessenger.Default.Register<GameStateChangedMessage>(this, (r, m) =>
-        //{
-        //    VisualStateManager.GoToElementState(MainGrid, m.GameMode.ToString(), true);
-        //});
+        WeakReferenceMessenger.Default.Register(this);
+        WeakReferenceMessenger.Default.UnregisterAllOnUnloaded(this);
     }
 
     public GamePageViewModel ViewModel
@@ -34,4 +31,13 @@ private async void Button_Click(object sender, RoutedEventArgs e)
     {
         await _navigationService.NavigateToAsync("TestPage");
     }
+
+    public void Receive(GameMoveMessage message)
+    {
+        if (message.GameMoveValue is not GameMoveValue.Completed)
+            return;
+
+        pegScrollViewer.UpdateLayout();
+        pegScrollViewer.ScrollToBottom();
+    }
 }

From 03703e0098fa0bcf59a6fa09b11649fd03547a55 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:10:00 +0100
Subject: [PATCH 3/9] Removed StackPanel to fix layout issues

---
 src/Codebreaker.WPF/Views/Pages/GamePage.xaml | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
index 5f503032..b898708d 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
@@ -15,9 +15,6 @@
         <conv:ColorNameToBrushConverter x:Key="ColorConverter" />
     </Page.Resources>
 
-    <StackPanel Orientation="Vertical">
-        <Button Click="Button_Click" Content="To TestPage" />
-
         <Grid>
         <Grid.RowDefinitions>
             <RowDefinition Height="Auto" />
@@ -109,5 +106,4 @@
             Minimum="0" Maximum="100" IsIndeterminate="True" 
             Visibility="{Binding ViewModel.InProgress, Converter={StaticResource BooleanToVisibilityConverter}}" />
         </Grid>
-    </StackPanel>
 </Page>

From 8f32344f01c00621e4a2548363dca2212fd1b9d8 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:10:26 +0100
Subject: [PATCH 4/9] Added TestPage button to bottom

---
 src/Codebreaker.WPF/Views/Pages/GamePage.xaml | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
index b898708d..cfc28b00 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
@@ -99,11 +99,19 @@
             ItemTemplate="{StaticResource PegsTemplate}" />
         </ScrollViewer>
 
-        <TextBlock Grid.Row="3" Text="{Binding ViewModel.GameStatus,Mode=OneWay}" />
+        <Grid Grid.Row="3">
+            <Grid.ColumnDefinitions>
+                <ColumnDefinition Width="Auto" />
+                <ColumnDefinition Width="*" />
+                <ColumnDefinition Width="Auto" />
+            </Grid.ColumnDefinitions>
+            <TextBlock Grid.Column="0" Text="{Binding ViewModel.GameStatus,Mode=OneWay}" />
             <ProgressBar
-            Grid.Row="3"
+                Grid.Column="1"
             Height="15"
             Minimum="0" Maximum="100" IsIndeterminate="True" 
             Visibility="{Binding ViewModel.InProgress, Converter={StaticResource BooleanToVisibilityConverter}}" />
+            <Button Grid.Column="2" Click="Button_Click" Content="To TestPage" Padding="15,2" />
+        </Grid>
         </Grid>
 </Page>

From 8986c0611796c3a83389e91193753cefe4b529dd Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:10:57 +0100
Subject: [PATCH 5/9] Disabled selection for peg ListView

---
 src/Codebreaker.WPF/Views/Pages/GamePage.xaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
index cfc28b00..5ac45b63 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
@@ -94,6 +94,8 @@
         <ScrollViewer Grid.Row="2" Padding="0,0,0,15" x:Name="pegScrollViewer">
         <ListView
             Height="Auto"
+                SelectedIndex="-1"
+                IsHitTestVisible="False"
             Visibility="{Binding ViewModel.GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibilityConverter}, ConverterParameter=Running}"
             ItemsSource="{Binding ViewModel.GameMoves, Mode=OneWay}"
             ItemTemplate="{StaticResource PegsTemplate}" />

From 318e79a7b74dd48a3c2e76bde03878062675c8da Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:11:22 +0100
Subject: [PATCH 6/9] Removed border for peg ListView and stretched items
 horizontally

---
 src/Codebreaker.WPF/Views/Pages/GamePage.xaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
index 5ac45b63..f57886a4 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
@@ -93,9 +93,10 @@
 
         <ScrollViewer Grid.Row="2" Padding="0,0,0,15" x:Name="pegScrollViewer">
         <ListView
-            Height="Auto"
+                BorderThickness="0"
                 SelectedIndex="-1"
                 IsHitTestVisible="False"
+                HorizontalContentAlignment="Stretch"
             Visibility="{Binding ViewModel.GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibilityConverter}, ConverterParameter=Running}"
             ItemsSource="{Binding ViewModel.GameMoves, Mode=OneWay}"
             ItemTemplate="{StaticResource PegsTemplate}" />

From ef9a44ee930cb7d08ec8e6e2d384d5f5b15908f8 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:11:58 +0100
Subject: [PATCH 7/9] Fixed indentations

---
 src/Codebreaker.WPF/Views/Pages/GamePage.xaml | 72 +++++++++----------
 1 file changed, 36 insertions(+), 36 deletions(-)

diff --git a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
index f57886a4..24921a62 100644
--- a/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.WPF/Views/Pages/GamePage.xaml
@@ -15,7 +15,7 @@
         <conv:ColorNameToBrushConverter x:Key="ColorConverter" />
     </Page.Resources>
 
-        <Grid>
+    <Grid>
         <Grid.RowDefinitions>
             <RowDefinition Height="Auto" />
             <RowDefinition Height="Auto" />
@@ -25,35 +25,35 @@
 
         <!--<StackPanel Orientation="Vertical" Grid.Row="0">
             --><!-- TODO: Create component for this, to make it (more) generic. --><!--
-            <Grid Visibility="{Binding Path=ViewModel.InfoMessage.IsVisible, Mode=OneWay, Converter={StaticResource BooleanToVisibilityConverter}}">
-                <Grid.ColumnDefinitions>
-                    <ColumnDefinition Width="Auto" />
-                    <ColumnDefinition Width="*" />
-                    <ColumnDefinition Width="Auto" />
-                </Grid.ColumnDefinitions>
+                <Grid Visibility="{Binding Path=ViewModel.InfoMessage.IsVisible, Mode=OneWay, Converter={StaticResource BooleanToVisibilityConverter}}">
+                    <Grid.ColumnDefinitions>
+                        <ColumnDefinition Width="Auto" />
+                        <ColumnDefinition Width="*" />
+                        <ColumnDefinition Width="Auto" />
+                    </Grid.ColumnDefinitions>
 
-                <TextBlock Text="Information: " VerticalAlignment="Center" Margin="8,0" />
-                <TextBlock Grid.Column="1" Text="{Binding ViewModel.InfoMessage.Message, Mode=OneWay}" VerticalAlignment="Center" Margin="8,0" />
-                <Button
-                    Grid.Column="2"
-                    Content="{Binding ViewModel.InfoMessage.ActionTitle, Mode=OneWay}" Padding="30,8"
-                    Command="{Binding ViewModel.InfoMessage.ActionCommand, Mode=OneWay}" />
-            </Grid>
-            <Grid Visibility="{Binding Path=ViewModel.ErrorMessage.IsVisible, Mode=OneWay, Converter={StaticResource BooleanToVisibilityConverter}}">
-                <Grid.ColumnDefinitions>
-                    <ColumnDefinition Width="Auto" />
-                    <ColumnDefinition Width="*" />
-                    <ColumnDefinition Width="Auto" />
-                </Grid.ColumnDefinitions>
+                    <TextBlock Text="Information: " VerticalAlignment="Center" Margin="8,0" />
+                    <TextBlock Grid.Column="1" Text="{Binding ViewModel.InfoMessage.Message, Mode=OneWay}" VerticalAlignment="Center" Margin="8,0" />
+                    <Button
+                        Grid.Column="2"
+                        Content="{Binding ViewModel.InfoMessage.ActionTitle, Mode=OneWay}" Padding="30,8"
+                        Command="{Binding ViewModel.InfoMessage.ActionCommand, Mode=OneWay}" />
+                </Grid>
+                <Grid Visibility="{Binding Path=ViewModel.ErrorMessage.IsVisible, Mode=OneWay, Converter={StaticResource BooleanToVisibilityConverter}}">
+                    <Grid.ColumnDefinitions>
+                        <ColumnDefinition Width="Auto" />
+                        <ColumnDefinition Width="*" />
+                        <ColumnDefinition Width="Auto" />
+                    </Grid.ColumnDefinitions>
 
-                <TextBlock Text="Error: " VerticalAlignment="Center" Margin="8,0" />
-                <TextBlock Grid.Column="1" Text="{Binding ViewModel.ErrorMessage.Message, Mode=OneWay}" VerticalAlignment="Center" Margin="8,0" />
-                <Button
-                    Grid.Column="2"
-                    Content="{Binding ViewModel.ErrorMessage.ActionTitle, Mode=OneWay}" Padding="30,8"
-                    Command="{Binding ViewModel.ErrorMessage.ActionCommand, Mode=OneWay}" />
-            </Grid>
-        </StackPanel>-->
+                    <TextBlock Text="Error: " VerticalAlignment="Center" Margin="8,0" />
+                    <TextBlock Grid.Column="1" Text="{Binding ViewModel.ErrorMessage.Message, Mode=OneWay}" VerticalAlignment="Center" Margin="8,0" />
+                    <Button
+                        Grid.Column="2"
+                        Content="{Binding ViewModel.ErrorMessage.ActionTitle, Mode=OneWay}" Padding="30,8"
+                        Command="{Binding ViewModel.ErrorMessage.ActionCommand, Mode=OneWay}" />
+                </Grid>
+            </StackPanel>-->
         <components:GameResultDisplay
             Panel.ZIndex="1"
             Grid.Row="0"/>
@@ -92,14 +92,14 @@
             ViewModel="{Binding ViewModel, ElementName=Self, Mode=OneWay}" />
 
         <ScrollViewer Grid.Row="2" Padding="0,0,0,15" x:Name="pegScrollViewer">
-        <ListView
+            <ListView
                 BorderThickness="0"
                 SelectedIndex="-1"
                 IsHitTestVisible="False"
                 HorizontalContentAlignment="Stretch"
-            Visibility="{Binding ViewModel.GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibilityConverter}, ConverterParameter=Running}"
-            ItemsSource="{Binding ViewModel.GameMoves, Mode=OneWay}"
-            ItemTemplate="{StaticResource PegsTemplate}" />
+                Visibility="{Binding ViewModel.GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibilityConverter}, ConverterParameter=Running}"
+                ItemsSource="{Binding ViewModel.GameMoves, Mode=OneWay}"
+                ItemTemplate="{StaticResource PegsTemplate}" />
         </ScrollViewer>
 
         <Grid Grid.Row="3">
@@ -111,10 +111,10 @@
             <TextBlock Grid.Column="0" Text="{Binding ViewModel.GameStatus,Mode=OneWay}" />
             <ProgressBar
                 Grid.Column="1"
-            Height="15"
-            Minimum="0" Maximum="100" IsIndeterminate="True" 
-            Visibility="{Binding ViewModel.InProgress, Converter={StaticResource BooleanToVisibilityConverter}}" />
+                Height="15"
+                Minimum="0" Maximum="100" IsIndeterminate="True" 
+                Visibility="{Binding ViewModel.InProgress, Converter={StaticResource BooleanToVisibilityConverter}}" />
             <Button Grid.Column="2" Click="Button_Click" Content="To TestPage" Padding="15,2" />
         </Grid>
-        </Grid>
+    </Grid>
 </Page>

From 76032fb838e9346f6f18b68b2ddd9e77b0d68aa6 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:12:25 +0100
Subject: [PATCH 8/9] Made items in peg ListView lightGray

---
 src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml b/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml
index 60252585..67d7314b 100644
--- a/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml
+++ b/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml
@@ -9,7 +9,7 @@
     <conv:IntToEnumerableConverter x:Key="IntToEnumerableConverter" />-->
 
     <DataTemplate x:Key="PegsTemplate">
-        <Grid Background="Transparent">
+        <Grid Background="LightGray">
             <Grid.ColumnDefinitions>
                 <ColumnDefinition Width="auto" />
                 <ColumnDefinition Width="auto" />

From 17d19276f853a671dd2f377f38baf514c4a83c9e Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Thu, 8 Feb 2024 13:12:34 +0100
Subject: [PATCH 9/9] Removed unnecessary XAML

---
 .../Views/Templates/CodeBreakerTemplates.xaml | 32 +++++++------------
 1 file changed, 12 insertions(+), 20 deletions(-)

diff --git a/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml b/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml
index 67d7314b..43e7f0a0 100644
--- a/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml
+++ b/src/Codebreaker.WPF/Views/Templates/CodeBreakerTemplates.xaml
@@ -4,9 +4,6 @@
 
     <conv:KeyPegColorNameToBrushConverter x:Key="KeyPegColorNameToBrushConverter" />
     <conv:ColorNameToBrushConverter x:Key="ColorConverter" />
-    <!--<conv:ColorNameToBrushConverter x:Key="ColorConverter" />
-    <conv:SelectionAndKeyPegToKeyVisibilityConverter x:Key="KeyPegVisibilityConverter" />
-    <conv:IntToEnumerableConverter x:Key="IntToEnumerableConverter" />-->
 
     <DataTemplate x:Key="PegsTemplate">
         <Grid Background="LightGray">
@@ -29,23 +26,18 @@
                 </ItemsControl.ItemTemplate>
             </ItemsControl>
 
-            <Grid Grid.Column="2">
-                <Grid.RowDefinitions>
-                    <RowDefinition />
-                </Grid.RowDefinitions>
-                <ItemsControl Grid.Row="0" ItemsSource="{Binding Path=KeyPegs, Mode=OneTime}">
-                    <ItemsControl.ItemsPanel>
-                        <ItemsPanelTemplate>
-                            <StackPanel Orientation="Horizontal" />
-                        </ItemsPanelTemplate>
-                    </ItemsControl.ItemsPanel>
-                    <ItemsControl.ItemTemplate>
-                        <DataTemplate>
-                            <Ellipse Width="16" Height="16" Fill="{Binding Converter={StaticResource KeyPegColorNameToBrushConverter}}" Margin="10,0" />
-                        </DataTemplate>
-                    </ItemsControl.ItemTemplate>
-                </ItemsControl>
-            </Grid>
+            <ItemsControl Grid.Column="2" ItemsSource="{Binding Path=KeyPegs, Mode=OneTime}">
+                <ItemsControl.ItemsPanel>
+                    <ItemsPanelTemplate>
+                        <StackPanel Orientation="Horizontal" />
+                    </ItemsPanelTemplate>
+                </ItemsControl.ItemsPanel>
+                <ItemsControl.ItemTemplate>
+                    <DataTemplate>
+                        <Ellipse Width="16" Height="16" Fill="{Binding Converter={StaticResource KeyPegColorNameToBrushConverter}}" Margin="10,0" />
+                    </DataTemplate>
+                </ItemsControl.ItemTemplate>
+            </ItemsControl>
         </Grid>
     </DataTemplate>
 </ResourceDictionary>