[X|C] only use a single GridLengthTypeConverter (#29376)

* [X|C] only use a single GridLengthTypeConverter

- fixes #29334

* fix tests

Author: StephaneDelcroix

src/Controls/Foldable/src/TwoPaneView.cs

@@ -5,6 +5,7 @@
 using Microsoft.Maui.Devices;
 using Microsoft.Maui.Foldable;
 using Microsoft.Maui.Graphics;
+using Microsoft.Maui.Converters;
 
 namespace Microsoft.Maui.Controls.Foldable
 {
@@ -152,7 +153,7 @@ public double MinWideModeWidth
 		/// <summary>
 		/// Gets the calculated width (in wide mode) or height (in tall mode) of pane 1, or sets the GridLength value of pane 1.
 		/// </summary>
-		[System.ComponentModel.TypeConverter(typeof(GridLengthTypeConverter))]
+		[System.ComponentModel.TypeConverter(typeof(Converters.GridLengthTypeConverter))]
 		public GridLength Pane1Length
 		{
 			get { return (GridLength)GetValue(Pane1LengthProperty); }
@@ -162,7 +163,7 @@ public GridLength Pane1Length
 		/// <summary>
 		/// Gets the calculated width (in wide mode) or height (in tall mode) of pane 2, or sets the GridLength value of pane 2.
 		/// </summary>
-		[System.ComponentModel.TypeConverter(typeof(GridLengthTypeConverter))]
+		[System.ComponentModel.TypeConverter(typeof(Converters.GridLengthTypeConverter))]
 		public GridLength Pane2Length
 		{
 			get { return (GridLength)GetValue(Pane2LengthProperty); }

src/Controls/src/Build.Tasks/XamlCache.cs

@@ -74,6 +74,8 @@ public ICollection<string> GetResourceNamesInUse(VariableDefinition variableDefi
 		{ module.ImportReference(this, ("Microsoft.Maui", "Microsoft.Maui.Converters", "FlexAlignSelfTypeConverter")), typeof(EnumTypeConverter<Layouts.FlexAlignSelf>) },
 		{ module.ImportReference(this, ("Microsoft.Maui", "Microsoft.Maui.Converters", "FlexWrapTypeConverter")), typeof(EnumTypeConverter<Layouts.FlexWrap>) },
 		{ module.ImportReference(this, ("Microsoft.Maui", "Microsoft.Maui.Converters", "FlexBasisTypeConverter")), typeof(FlexBasisTypeConverter) },
+		{ module.ImportReference(this, ("Microsoft.Maui", "Microsoft.Maui.Converters", "GridLengthTypeConverter")), typeof(Microsoft.Maui.Controls.XamlC.GridLengthTypeConverter) },
+
 	};
 
 	// State used by SetPropertiesVisitor

src/Controls/src/Core/ColumnDefinition.cs

@@ -19,7 +19,7 @@ public ColumnDefinition(GridLength width)
 			=> SetValue(WidthProperty, width);
 
 		/// <summary>Gets or sets the width of the column.</summary>
-		[System.ComponentModel.TypeConverter(typeof(GridLengthTypeConverter))]
+		[System.ComponentModel.TypeConverter(typeof(Converters.GridLengthTypeConverter))]
 		public GridLength Width
 		{
 			get { return (GridLength)GetValue(WidthProperty); }

src/Controls/src/Core/ColumnDefinitionCollectionTypeConverter.cs

@@ -33,7 +33,7 @@ public override bool CanConvertTo(ITypeDescriptorContext? context, Type? destina
 			var definitions = new List<ColumnDefinition>(count);
 			foreach (var range in unsplit.Split(','))
 			{
-				var length = GridLengthTypeConverter.ParseStringToGridLength(unsplit[range]);
+				var length = Converters.GridLengthTypeConverter.ParseStringToGridLength(unsplit[range]);
 				definitions.Add(new ColumnDefinition(length));
 			}
 #else
@@ -42,7 +42,7 @@ public override bool CanConvertTo(ITypeDescriptorContext? context, Type? destina
 			var definitions = new List<ColumnDefinition>(count);
 			foreach (var lengthStr in lengths)
 			{
-				var length = GridLengthTypeConverter.ParseStringToGridLength(lengthStr);
+				var length = Converters.GridLengthTypeConverter.ParseStringToGridLength(lengthStr);
 				definitions.Add(new ColumnDefinition(length));
 			}
 #endif
@@ -62,15 +62,15 @@ public override bool CanConvertTo(ITypeDescriptorContext? context, Type? destina
 			if (count == 0)
 				return string.Empty;
 			if (count == 1)
-				return GridLengthTypeConverter.ConvertToString(definitions[0].Width);
+				return Converters.GridLengthTypeConverter.ConvertToString(definitions[0].Width);
 
 			// for multiple items
 			var pool = ArrayPool<string>.Shared;
 			var rentedArray = pool.Rent(definitions.Count);
 			for (var i = 0; i < definitions.Count; i++)
 			{
 				var definition = definitions[i];
-				rentedArray[i] = GridLengthTypeConverter.ConvertToString(definition.Width);
+				rentedArray[i] = Converters.GridLengthTypeConverter.ConvertToString(definition.Width);
 			}
 			var result = string.Join(", ", rentedArray, 0, definitions.Count);
 			pool.Return(rentedArray);

src/Controls/src/Core/GridLengthTypeConverter.cs

@@ -6,6 +6,7 @@
 namespace Microsoft.Maui.Controls
 {
 	[ProvideCompiled("Microsoft.Maui.Controls.XamlC.GridLengthTypeConverter")]
+	[Obsolete("Microsoft.Maui.Controls.GridLengthTypeConverter is obsolete. Use Microsoft.Maui.Converters.GridLengthConverter instead.")]
 	public class GridLengthTypeConverter : TypeConverter
 	{
 		public override bool CanConvertFrom(ITypeDescriptorContext? context, Type sourceType)

src/Controls/src/Core/RowDefinition.cs

@@ -21,7 +21,7 @@ public RowDefinition(GridLength height)
 		}
 
 		/// <include file="../../docs/Microsoft.Maui.Controls/RowDefinition.xml" path="//Member[@MemberName='Height']/Docs/*" />
-		[System.ComponentModel.TypeConverter(typeof(GridLengthTypeConverter))]
+		[System.ComponentModel.TypeConverter(typeof(Converters.GridLengthTypeConverter))]
 		public GridLength Height
 		{
 			get { return (GridLength)GetValue(HeightProperty); }

src/Controls/src/Core/RowDefinitionCollectionTypeConverter.cs

@@ -33,7 +33,7 @@ public override bool CanConvertTo(ITypeDescriptorContext? context, Type? destina
 			var definitions = new List<RowDefinition>(count);
 			foreach (var range in unsplit.Split(','))
 			{
-				var length = GridLengthTypeConverter.ParseStringToGridLength(unsplit[range]);
+				var length = Converters.GridLengthTypeConverter.ParseStringToGridLength(unsplit[range]);
 				definitions.Add(new RowDefinition(length));
 			}
 #else
@@ -42,7 +42,7 @@ public override bool CanConvertTo(ITypeDescriptorContext? context, Type? destina
 			var definitions = new List<RowDefinition>(count);
 			foreach (var lengthStr in lengths)
 			{
-				var length = GridLengthTypeConverter.ParseStringToGridLength(lengthStr);
+				var length = Converters.GridLengthTypeConverter.ParseStringToGridLength(lengthStr);
 				definitions.Add(new RowDefinition(length));
 			}
 #endif
@@ -61,15 +61,15 @@ public override bool CanConvertTo(ITypeDescriptorContext? context, Type? destina
 			if (count == 0)
 				return string.Empty;
 			if (count == 1)
-				return GridLengthTypeConverter.ConvertToString(definitions[0].Height);
+				return Converters.GridLengthTypeConverter.ConvertToString(definitions[0].Height);
 
 			// for multiple items
 			var pool = ArrayPool<string>.Shared;
 			var rentedArray = pool.Rent(definitions.Count);
 			for (var i = 0; i < definitions.Count; i++)
 			{
 				var definition = definitions[i];
-				rentedArray[i] = GridLengthTypeConverter.ConvertToString(definition.Height);
+				rentedArray[i] = Converters.GridLengthTypeConverter.ConvertToString(definition.Height);
 			}
 			var result = string.Join(", ", rentedArray, 0, definitions.Count);
 			pool.Return(rentedArray);

src/Controls/src/SourceGen/NodeSGExtensions.cs

@@ -50,7 +50,6 @@ private static string ConvertEnum(string value, BaseNode node, ITypeSymbol toTyp
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Graphics.Converters.RectTypeConverter")!, (CreateLazyConverter<RectConverter>(), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Graphics.Rect")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Graphics.Converters.ColorTypeConverter")!, (CreateLazyConverter<ColorConverter>(), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Graphics.Color")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Graphics.Converters.PointTypeConverter")!, (CreateLazyConverter<PointConverter>(), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Graphics.Point")!) },
-		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.ThicknessTypeConverter")!, (CreateLazyConverter<ThicknessConverter>(), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Thickness")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.CornerRadiusTypeConverter")!, (CreateLazyRegistryConverter("Microsoft.Maui.CornerRadius"), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.CornerRadius")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.EasingTypeConverter")!, (CreateLazyRegistryConverter("Microsoft.Maui.Easing"), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Easing")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.FlexJustifyTypeConverter")!, (ConvertEnum, context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Layouts.FlexJustify")!) },
@@ -60,6 +59,8 @@ private static string ConvertEnum(string value, BaseNode node, ITypeSymbol toTyp
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.FlexAlignSelfTypeConverter")!, (ConvertEnum, context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Layouts.FlexAlignSelf")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.FlexWrapTypeConverter")!, (ConvertEnum, context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Layouts.FlexWrap")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.FlexBasisTypeConverter")!, (CreateLazyRegistryConverter("Microsoft.Maui.Layouts.FlexBasis"), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Layouts.FlexBasis")!) },
+		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.GridLengthTypeConverter")!, (CreateLazyConverter<GridLengthConverter>(), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.GridLength")!) },
+		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Converters.ThicknessTypeConverter")!, (CreateLazyConverter<ThicknessConverter>(), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Thickness")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Controls.ColumnDefinitionCollectionTypeConverter")!, (CreateLazyRegistryConverter("Microsoft.Maui.Controls.ColumnDefinitionCollection"), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Controls.ColumnDefinitionCollection")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Controls.FlowDirectionConverter")!, (CreateLazyRegistryConverter("Microsoft.Maui.FlowDirection"), context.Compilation.GetTypeByMetadataName("Microsoft.Maui.FlowDirection")!) },
 		{ context.Compilation.GetTypeByMetadataName("Microsoft.Maui.Controls.FontSizeConverter")!, (CreateLazyRegistryConverter("System.Double"), context.Compilation.GetTypeByMetadataName("System.Double")!) },

src/Controls/tests/Xaml.UnitTests/Issues/Maui29334.xaml

@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
+             xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
+             x:Class="Microsoft.Maui.Controls.Xaml.UnitTests.Maui29334"
+             Title="Maui29334">
+    <Grid x:Name="grid">
+        <Grid.RowDefinitions>
+            <RowDefinition>
+                <RowDefinition.Height>
+                    <OnIdiom x:TypeArguments="GridLength" Default="50" Desktop="100"/>
+                </RowDefinition.Height>
+            </RowDefinition>
+            <RowDefinition Height="*" />
+        </Grid.RowDefinitions>
+    
+        <Label Text="Hello, World!" />
+    </Grid>
+
+</ContentPage>

src/Controls/tests/Xaml.UnitTests/Issues/Maui29334.xaml.cs

@@ -0,0 +1,38 @@
+namespace Microsoft.Maui.Controls.Xaml.UnitTests;
+
+using Microsoft.Maui.ApplicationModel;
+using Microsoft.Maui.Controls.Core.UnitTests;
+using Microsoft.Maui.Dispatching;
+using Microsoft.Maui.UnitTests;
+using NUnit.Framework;
+
+public partial class Maui29334 : ContentPage
+{
+
+	public Maui29334() => InitializeComponent();
+
+	public Maui29334(bool useCompiledXaml)
+	{
+		//this stub will be replaced at compile time
+	}
+
+	[TestFixture]
+	class Test
+	{
+		[SetUp]
+		public void Setup()
+		{
+			Application.SetCurrentApplication(new MockApplication());
+			DispatcherProvider.SetCurrent(new DispatcherProviderStub());
+		}
+
+		[TearDown] public void TearDown() => AppInfo.SetCurrent(null);
+
+		[Test]
+		public void OnIdiomGridLength([Values] bool useCompiledXaml)
+		{
+			var page = new Maui29334(useCompiledXaml);
+			
+		}
+	}
+}