From 2091039fd18dffa96b92e4c3cc9f9dcf453f0e9c Mon Sep 17 00:00:00 2001
From: David Britch <davidbritch@users.noreply.github.com>
Date: Mon, 22 Jan 2024 17:13:08 +0000
Subject: [PATCH 1/3] Formating changes.

---
 .../CustomLayoutManagerFactory.cs             | 22 +++++++-------
 .../Layouts/CustomGridLayoutManager.cs        |  2 +-
 .../CustomLayoutDemos/MauiProgram.cs          | 30 +++++++++----------
 .../Views/CustomizedGridPage.xaml             |  4 +--
 4 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/CustomLayoutManagerFactory.cs b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/CustomLayoutManagerFactory.cs
index 33a79ebd2..ae8d4765a 100644
--- a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/CustomLayoutManagerFactory.cs
+++ b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/CustomLayoutManagerFactory.cs
@@ -3,16 +3,16 @@
 
 namespace CustomLayoutDemos
 {
-	public class CustomLayoutManagerFactory : ILayoutManagerFactory
-	{
-		public ILayoutManager CreateLayoutManager(Layout layout)
-		{
-			if (layout is Grid)
-			{
-				return new CustomGridLayoutManager(layout as IGridLayout);
-			}
-			return null;
-		}
-	}
+    public class CustomLayoutManagerFactory : ILayoutManagerFactory
+    {
+        public ILayoutManager CreateLayoutManager(Layout layout)
+        {
+            if (layout is Grid)
+            {
+                return new CustomGridLayoutManager(layout as IGridLayout);
+            }
+            return null;
+        }
+    }
 }
 
diff --git a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/CustomGridLayoutManager.cs b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/CustomGridLayoutManager.cs
index 6b84c5c08..3234c3202 100644
--- a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/CustomGridLayoutManager.cs
+++ b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/CustomGridLayoutManager.cs
@@ -28,7 +28,7 @@ void EnsureRows()
                 maxRow = Math.Max(grid.GetRow(child), maxRow);
             }
 
-            // Add more rows if we need them
+            // Add more rows if needed
             for (int n = grid.RowDefinitions.Count; n <= maxRow; n++)
             {
                 grid.RowDefinitions.Add(new RowDefinition(GridLength.Star));
diff --git a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/MauiProgram.cs b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/MauiProgram.cs
index 082ca1e75..fadc71ba3 100644
--- a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/MauiProgram.cs
+++ b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/MauiProgram.cs
@@ -4,25 +4,25 @@ namespace CustomLayoutDemos;
 
 public static class MauiProgram
 {
-	public static MauiApp CreateMauiApp()
-	{
-		var builder = MauiApp.CreateBuilder();
-		builder
-			.UseMauiApp<App>()
-			.ConfigureFonts(fonts =>
-			{
-				fonts.AddFont("OpenSans-Regular.ttf", "OpenSansRegular");
-				fonts.AddFont("OpenSans-Semibold.ttf", "OpenSansSemibold");
-			});
+    public static MauiApp CreateMauiApp()
+    {
+        var builder = MauiApp.CreateBuilder();
+        builder
+            .UseMauiApp<App>()
+            .ConfigureFonts(fonts =>
+            {
+                fonts.AddFont("OpenSans-Regular.ttf", "OpenSansRegular");
+                fonts.AddFont("OpenSans-Semibold.ttf", "OpenSansSemibold");
+            });
 
-		// Setup a custom layout manager so the default manager for the Grid can be replaced.
-		builder.Services.Add(new ServiceDescriptor(typeof(ILayoutManagerFactory), new CustomLayoutManagerFactory()));
+        // Setup a custom layout manager so the default manager for the Grid can be replaced.
+        builder.Services.Add(new ServiceDescriptor(typeof(ILayoutManagerFactory), new CustomLayoutManagerFactory()));
 
 #if DEBUG
-		builder.Logging.AddDebug();
+        builder.Logging.AddDebug();
 #endif
 
-		return builder.Build();
-	}
+        return builder.Build();
+    }
 }
 
diff --git a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Views/CustomizedGridPage.xaml b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Views/CustomizedGridPage.xaml
index eb6261be9..8646a7c5f 100644
--- a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Views/CustomizedGridPage.xaml
+++ b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Views/CustomizedGridPage.xaml
@@ -10,8 +10,8 @@
         <Label Grid.Row="2"
                Text="Notice that the Grid doesn't explicitly specify a RowDefinitions collection." />
         <Label Grid.Row="3"
-               Text="In MauiProgram.cs, we've added an instance of an ILayoutManagerFactory which replaces the default GridLayoutManager. The custom manager will automatically add the necessary RowDefinitions at runtime." />
+               Text="In MauiProgram.cs, an instance of an ILayoutManagerFactory has been added that replaces the default GridLayoutManager. The custom manager will automatically add the necessary RowDefinitions at runtime." />
         <Label Grid.Row="5"
-               Text="We can even skip some rows, and it will add the intervening ones for us. (Notice the gap between the previous label and this one.)" />
+               Text="We can even skip some rows, and it will add the intervening ones for us (notice the gap between the previous label and this one)." />
     </Grid>
 </ContentPage>

From 7591ff1b4bece6b27346dd147f673603aefbf997 Mon Sep 17 00:00:00 2001
From: David Britch <davidbritch@users.noreply.github.com>
Date: Tue, 23 Jan 2024 12:18:30 +0000
Subject: [PATCH 2/3] Minor edits.

---
 .../CustomLayoutDemos/Layouts/HorizontalWrapLayout.cs    | 3 +--
 .../Layouts/HorizontalWrapLayoutManager.cs               | 9 ++++-----
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayout.cs b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayout.cs
index ca0227d5f..f60958f10 100644
--- a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayout.cs
+++ b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayout.cs
@@ -2,7 +2,7 @@
 
 namespace CustomLayoutDemos.Layouts
 {
-    public class HorizontalWrapLayout : StackLayout
+    public class HorizontalWrapLayout : HorizontalStackLayout
     {
         protected override ILayoutManager CreateLayoutManager()
         {
@@ -10,4 +10,3 @@ protected override ILayoutManager CreateLayoutManager()
         }
     }
 }
-
diff --git a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayoutManager.cs b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayoutManager.cs
index 7c80182d9..ee0c924e6 100644
--- a/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayoutManager.cs
+++ b/8.0/UserInterface/CustomLayoutDemos/CustomLayoutDemos/Layouts/HorizontalWrapLayoutManager.cs
@@ -1,9 +1,9 @@
 using Microsoft.Maui.Layouts;
-using StackLayoutManager = Microsoft.Maui.Layouts.StackLayoutManager;
+using HorizontalStackLayoutManager = Microsoft.Maui.Layouts.HorizontalStackLayoutManager;
 
 namespace CustomLayoutDemos.Layouts
 {
-    public class HorizontalWrapLayoutManager : StackLayoutManager
+    public class HorizontalWrapLayoutManager : HorizontalStackLayoutManager
     {
         HorizontalWrapLayout _layout;
 
@@ -26,7 +26,6 @@ public override Size Measure(double widthConstraint, double heightConstraint)
             for (int n = 0; n < _layout.Count; n++)
             {
                 var child = _layout[n];
-
                 if (child.Visibility == Visibility.Collapsed)
                 {
                     continue;
@@ -65,13 +64,13 @@ public override Size Measure(double widthConstraint, double heightConstraint)
             totalWidth += padding.HorizontalThickness;
             totalHeight += padding.VerticalThickness;
 
+            // Ensure that the total size of the layout fits within its constraints
             var finalWidth = ResolveConstraints(widthConstraint, Stack.Width, totalWidth, Stack.MinimumWidth, Stack.MaximumWidth);
             var finalHeight = ResolveConstraints(heightConstraint, Stack.Height, totalHeight, Stack.MinimumHeight, Stack.MaximumHeight);
 
             return new Size(finalWidth, finalHeight);
         }
 
-
         public override Size ArrangeChildren(Rect bounds)
         {
             var padding = Stack.Padding;
@@ -87,7 +86,6 @@ public override Size ArrangeChildren(Rect bounds)
             for (int n = 0; n < _layout.Count; n++)
             {
                 var child = _layout[n];
-
                 if (child.Visibility == Visibility.Collapsed)
                 {
                     continue;
@@ -113,6 +111,7 @@ public override Size ArrangeChildren(Rect bounds)
 
             var actual = new Size(maxStackWidth, currentRowTop + currentRowHeight);
 
+            // Adjust the size if the layout is set to fill its container
             return actual.AdjustForFill(bounds, Stack);
         }
     }

From 64b0004e90b5cdaa129e125ba4dbe42759d9afc2 Mon Sep 17 00:00:00 2001
From: Gerald Versluis <gerald.versluis@microsoft.com>
Date: Tue, 23 Jan 2024 13:45:41 +0100
Subject: [PATCH 3/3] Update build-pr.yml

---
 .github/workflows/build-pr.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/build-pr.yml b/.github/workflows/build-pr.yml
index 8589c2674..b2f3ace7e 100644
--- a/.github/workflows/build-pr.yml
+++ b/.github/workflows/build-pr.yml
@@ -31,7 +31,7 @@ jobs:
       # if: runner.os == 'macOS' # reenable when .NET 8 is default on hosted runners
 
     - name: Select Xcode Version
-      run: sudo xcode-select -s /Applications/Xcode_15.0.1.app
+      run: sudo xcode-select -s /Applications/Xcode_15.2.app
       if: runner.os == 'macOS' # Remove when Xcode 15+ is default on the hosted runners
 
     - name: Find and build changed projects