Skip to content

Comments

[Housekeeping] Port latest fixes from Xamarin.Forms to .NET MAUI#11420

Merged
PureWeen merged 13 commits intomainfrom
port-formschanges-I
Mar 9, 2023
Merged

[Housekeeping] Port latest fixes from Xamarin.Forms to .NET MAUI#11420
PureWeen merged 13 commits intomainfrom
port-formschanges-I

Conversation

@jsuarezruiz
Copy link
Contributor

Description of Change

Port latest fixes from Xamarin.Forms to .NET MAUI.

Issues Fixed

Fixes #10128

Copy link
Member

@jfversluis jfversluis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there is build errors here!

Copy link
Member

@rmarinho rmarinho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Core\Compatibility.ControlGallery.Core.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Core\Compatibility.ControlGallery.Core.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(15,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(16,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(15,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(16,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(15,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(16,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(15,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(16,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(18,28): error CS0246: The type or namespace name 'TestShell' could not be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(15,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(16,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(15,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(16,28): error CS0103: The name 'UITestCategories' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
    271 Warning(s)
    20 Error(s)

@rmarinho
Copy link
Member

Still failing

D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(51,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Core\Compatibility.ControlGallery.Core.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(54,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Core\Compatibility.ControlGallery.Core.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(51,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(54,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\Android.UITests\Compatibility.ControlGallery.Android.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(51,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(54,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\iOS.UITests\Compatibility.ControlGallery.iOS.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15542.cs(51,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
D:\a\_work\1\s\src\Compatibility\ControlGallery\src\Issues.Shared\Issue15565.cs(54,26): error CS0103: The name 'Color' does not exist in the current context [D:\a\_work\1\s\src\Compatibility\ControlGallery\test\WinUI.UITests\WinUI.UITests.csproj]
    271 Warning(s)
    8 Error(s)

@jsuarezruiz jsuarezruiz requested a review from PureWeen February 15, 2023 13:24
SearchHandler = Shell.GetSearchHandler(Page);
}
else if (e.PropertyName == Shell.TitleViewProperty.PropertyName)
else if (e.PropertyName == Shell.TitleViewProperty.PropertyName || e.PropertyName == VisualElement.HeightProperty.PropertyName || e.PropertyName == VisualElement.WidthProperty.PropertyName)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we change this to use e.IsOneOf ?

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2023

Thank you for your pull request. We are auto-formatting your source code to follow our code guidelines.

@PureWeen PureWeen enabled auto-merge March 7, 2023 22:00
@PureWeen PureWeen merged commit a775a2d into main Mar 9, 2023
@PureWeen PureWeen deleted the port-formschanges-I branch March 9, 2023 13:34
@github-actions github-actions bot locked and limited conversation to collaborators Dec 15, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! t/housekeeping ♻︎

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Shell.TitleView does not render on iOS 16

6 participants