From f1257a13dc992c3fb60186b7ba986ac3906158a6 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Mon, 12 Feb 2024 12:33:15 +0100
Subject: [PATCH 1/5] Removed "Header" font-size and added a small spacing

---
 src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml b/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml
index 1809177..c51cb59 100644
--- a/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml
+++ b/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml
@@ -12,8 +12,8 @@
     <ContentView.Resources>
         <conv:DictionaryToValuesConverter x:Key="DictionaryToValuesConverter" />
     </ContentView.Resources>
-    <VerticalStackLayout>
-        <Label Text="Select your colors" FontSize="Header" />
+    <VerticalStackLayout Spacing="5">
+        <Label Text="Select your colors" />
         <HorizontalStackLayout
             BindableLayout.ItemsSource="{Binding Path=Fields, Mode=OneWay}"
             Spacing="4">

From 6cd47285005af572f6afa473b79cd664ec64ddeb Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Mon, 12 Feb 2024 12:33:35 +0100
Subject: [PATCH 2/5] Removed "Header" font-size and improved layout

---
 .../Views/Pages/GamePage.xaml                 | 23 ++++++++++---------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml b/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
index 4fd4289..9560dc3 100644
--- a/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
@@ -26,20 +26,19 @@
             IsVisible="{Binding GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibleConverter}, ConverterParameter=Start}">
             <VerticalStackLayout
                 FlexLayout.Basis="500"
+                Padding="5,0"
                 FlexLayout.Grow="2">
-                <Label
-                    VerticalTextAlignment="Center"
-                    Text="Enter your Gamername" 
-                    FontSize="Header" />
+                <Label Text="Enter your player name" />
                 <Entry
                     IsEnabled="{Binding IsNameEnterable, Mode=OneWay}"
                     Text="{Binding Name, Mode=TwoWay }"
-                    Placeholder="Enter your name"/>
+                    Placeholder="Enter your player name"/>
             </VerticalStackLayout>
-            <Grid 
+            <Grid
                 FlexLayout.Basis="300"
                 FlexLayout.Grow="1"
                 Margin="0,5,0,0"
+                Padding="5,0"
                 ColumnDefinitions="*, auto"
                 MinimumWidthRequest="250"
                 IsVisible="{Binding GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibleConverter}, ConverterParameter=Start}">
@@ -63,14 +62,16 @@
             IsVisible="{Binding GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibleConverter}, ConverterParameter=Cancelable}"
             Wrap="Wrap"
             JustifyContent="Center"
-            AlignItems="Center">
-            <components:PegSelectionView />
+            AlignItems="End">
+            <components:PegSelectionView Padding="5,0" VerticalOptions="End" />
             <Grid
                 FlexLayout.Basis="200"
                 FlexLayout.Grow="1"
                 ColumnDefinitions="*, auto"
-                Padding="0,10"
-                MinimumWidthRequest="250">
+                MinimumWidthRequest="250"
+                VerticalOptions="End"
+                Padding="5,0"
+                Margin="0,5,0,0">
                 <Button
                     Grid.Column="0"
                     Text="Move"
@@ -86,7 +87,7 @@
             </Grid>
         </FlexLayout>
 
-        <ScrollView Grid.Row="2" Padding="0,0,0,15" x:Name="pegScrollView" HorizontalOptions="CenterAndExpand">
+        <ScrollView Grid.Row="2" Padding="0,15,0,15" x:Name="pegScrollView" HorizontalOptions="CenterAndExpand">
             <ListView
                 x:Name="test"
                 HorizontalOptions="FillAndExpand"

From 4526fee40be3b19f1ce7cd628ff10cdb5f70bf7c Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Mon, 12 Feb 2024 12:43:50 +0100
Subject: [PATCH 3/5] Reduced bottom margin

---
 src/Codebreaker.MAUI/Views/Components/GameResultDisplay.xaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Codebreaker.MAUI/Views/Components/GameResultDisplay.xaml b/src/Codebreaker.MAUI/Views/Components/GameResultDisplay.xaml
index 38e5ebe..2cde639 100644
--- a/src/Codebreaker.MAUI/Views/Components/GameResultDisplay.xaml
+++ b/src/Codebreaker.MAUI/Views/Components/GameResultDisplay.xaml
@@ -11,12 +11,12 @@
         <VerticalStackLayout IsVisible="{Binding GameStatus, Mode=OneWay, Converter={converter:GameStatusToIsVisibleConverter}, ConverterParameter=Won}">
             <Image Source="wonanimation_300_opt.gif" IsAnimationPlaying="True" MaximumHeightRequest="300" />
             <Label Text="Congratulations - you won the game 🎉🏆" FontSize="20" HorizontalOptions="Center" Margin="0,50,0,20" />
-            <Button Text="Great, play again" HorizontalOptions="Center" Margin="0,0,0,100" Command="{Binding StartGameCommand, Mode=OneTime}" />
+            <Button Text="Great, play again" HorizontalOptions="Center" Margin="0,0,0,50" Command="{Binding StartGameCommand, Mode=OneTime}" />
         </VerticalStackLayout>
         <VerticalStackLayout IsVisible="{Binding GameStatus, Mode=OneWay, Converter={converter:GameStatusToIsVisibleConverter}, ConverterParameter=Lost}">
             <Image Source="lostanimation_300_opt.gif" IsAnimationPlaying="True" MaximumHeightRequest="300" />
             <Label Text="Oh no - you lost the game 😑" FontSize="20" HorizontalOptions="Center" Margin="0,50,0,20" />
-            <Button Text="Ok, try again" HorizontalOptions="Center" Margin="0,0,0,100" Command="{Binding StartGameCommand, Mode=OneTime}" />
+            <Button Text="Ok, try again" HorizontalOptions="Center" Margin="0,0,0,50" Command="{Binding StartGameCommand, Mode=OneTime}" />
         </VerticalStackLayout>
     </Grid>
 </ContentView>

From 15726e18cc72a237ba911edd1a23e912e41bd895 Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Mon, 12 Feb 2024 12:44:05 +0100
Subject: [PATCH 4/5] Added spacing

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

diff --git a/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml b/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
index 9560dc3..70ede98 100644
--- a/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
@@ -27,7 +27,8 @@
             <VerticalStackLayout
                 FlexLayout.Basis="500"
                 Padding="5,0"
-                FlexLayout.Grow="2">
+                FlexLayout.Grow="2"
+                Spacing="5">
                 <Label Text="Enter your player name" />
                 <Entry
                     IsEnabled="{Binding IsNameEnterable, Mode=OneWay}"

From 5886ad8328d538c4e66aebb081f00175f287963a Mon Sep 17 00:00:00 2001
From: Sebastian Szvetecz <sebastian@szvetecz.at>
Date: Mon, 12 Feb 2024 12:44:24 +0100
Subject: [PATCH 5/5] Moved move submit button to PegSelectionView

---
 .../Views/Components/PegSelectionView.xaml    | 45 ++++++++++++++-----
 .../Views/Pages/GamePage.xaml                 | 32 ++-----------
 2 files changed, 38 insertions(+), 39 deletions(-)

diff --git a/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml b/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml
index c51cb59..fa0a384 100644
--- a/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml
+++ b/src/Codebreaker.MAUI/Views/Components/PegSelectionView.xaml
@@ -12,22 +12,47 @@
     <ContentView.Resources>
         <conv:DictionaryToValuesConverter x:Key="DictionaryToValuesConverter" />
     </ContentView.Resources>
-    <VerticalStackLayout Spacing="5">
-        <Label Text="Select your colors" />
-        <HorizontalStackLayout
+    <FlexLayout
+        Wrap="Wrap"
+        JustifyContent="Center"
+        AlignItems="End">
+        <VerticalStackLayout Spacing="5" Padding="5,0" VerticalOptions="End">
+            <Label Text="Select your colors" />
+            <HorizontalStackLayout
             BindableLayout.ItemsSource="{Binding Path=Fields, Mode=OneWay}"
             Spacing="4">
-            <BindableLayout.ItemTemplate>
-                <DataTemplate>
-                    <Picker
+                <BindableLayout.ItemTemplate>
+                    <DataTemplate>
+                        <Picker
                         x:Name="PegSelectionPicker"
                         x:DataType="{x:Type vmComponents:SelectedFieldViewModel}"
                         ItemsSource="{Binding Path=BindingContext.Game.FieldValues, Converter={StaticResource DictionaryToValuesConverter}, ConverterParameter='colors', Mode=OneWay, Source={x:Reference PegSelectionViewControl}}"
                         SelectedItem="{Binding Value, Mode=TwoWay}"
                         MinimumWidthRequest="50"
                         WidthRequest="88" />
-                </DataTemplate>
-            </BindableLayout.ItemTemplate>
-        </HorizontalStackLayout>
-    </VerticalStackLayout>
+                    </DataTemplate>
+                </BindableLayout.ItemTemplate>
+            </HorizontalStackLayout>
+        </VerticalStackLayout>
+        <Grid
+            FlexLayout.Basis="200"
+            FlexLayout.Grow="1"
+            ColumnDefinitions="*, auto"
+            MinimumWidthRequest="250"
+            VerticalOptions="End"
+            Padding="5,0"
+            Margin="0,5,0,0">
+            <Button
+                Grid.Column="0"
+                Text="Move"
+                MinimumWidthRequest="150"
+                Command="{Binding SetMoveCommand, Mode=OneTime}" />
+            <ActivityIndicator
+                Grid.Column="1"
+                Margin="15,0,0,0"
+                IsVisible="{Binding InProgress, Mode=OneWay}"
+                IsRunning="True"
+                VerticalOptions="Center" />
+        </Grid>
+    </FlexLayout>
 </ContentView>
diff --git a/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml b/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
index 70ede98..810dd45 100644
--- a/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
+++ b/src/Codebreaker.MAUI/Views/Pages/GamePage.xaml
@@ -57,37 +57,11 @@
             </Grid>
         </FlexLayout>
 
-        <FlexLayout
+        <components:PegSelectionView
             Grid.Row="1"
             MaximumWidthRequest="700"
-            IsVisible="{Binding GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibleConverter}, ConverterParameter=Cancelable}"
-            Wrap="Wrap"
-            JustifyContent="Center"
-            AlignItems="End">
-            <components:PegSelectionView Padding="5,0" VerticalOptions="End" />
-            <Grid
-                FlexLayout.Basis="200"
-                FlexLayout.Grow="1"
-                ColumnDefinitions="*, auto"
-                MinimumWidthRequest="250"
-                VerticalOptions="End"
-                Padding="5,0"
-                Margin="0,5,0,0">
-                <Button
-                    Grid.Column="0"
-                    Text="Move"
-                    MinimumWidthRequest="150"
-                    IsVisible="{Binding GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibleConverter}, ConverterParameter=Cancelable}"
-                    Command="{Binding SetMoveCommand, Mode=OneTime}" />
-                <ActivityIndicator
-                    Grid.Column="1"
-                    Margin="15,0,0,0"
-                    IsVisible="{Binding InProgress, Mode=OneWay}"
-                    IsRunning="True"
-                    VerticalOptions="Center" />
-            </Grid>
-        </FlexLayout>
-
+            IsVisible="{Binding GameStatus, Mode=OneWay, Converter={StaticResource GameStatusVisibleConverter}, ConverterParameter=Cancelable}"/>
+        
         <ScrollView Grid.Row="2" Padding="0,15,0,15" x:Name="pegScrollView" HorizontalOptions="CenterAndExpand">
             <ListView
                 x:Name="test"