-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove unsupported targets in Device class #4491
Conversation
The whole UWP to WinUI makes me think about the compatibility story. Everyone will need to update all their XAML for this. |
Whatever we do here, we need to match in the essentials DeviceInfo API because this is going to use that eventually with #1965 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unitests are failing
Pending questions:
|
|
We can maybe remove UWP in net7 |
@Redth Maintain UWP until .NET 7? |
mark UWP as Obsolete |
src/Controls/tests/Xaml.UnitTests/MarkupExpressionParserTests.cs
Outdated
Show resolved
Hide resolved
@@ -94,6 +94,51 @@ public void TypeConverterAndDerivedTypes() | |||
image = new Image().LoadFromXaml(xaml); | |||
Assert.AreEqual("icon_twitter.png", (image.Source as FileImageSource).File); | |||
} | |||
|
|||
[Test] | |||
public void UWPisWinUI() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
* Implement IBorder on Windows (#5008) * Implement IBorder on Windows * Fix rectangles Co-authored-by: redth <jondick@gmail.com> * Remove all usage of Device throughout the repo (#4982) * Remove unsupported targets in Device class (#4491) * Remove unsupported targets in Device class * Fixed broken tests * Deprecate UWP target * Allow to use UWP target on XAML * Fix build errors * comments * UWP == WinUI * Fix Build errors * Added more tests * fix * vbnm * More fixes * things Co-authored-by: Matthew Leibowitz <mattleibow@live.com> Co-authored-by: Javier Suárez <javiersuarezruiz@hotmail.com> Co-authored-by: Matthew Leibowitz <mattleibow@live.com>
* Implement IBorder on Windows (#5008) * Implement IBorder on Windows * Fix rectangles Co-authored-by: redth <jondick@gmail.com> * Remove all usage of Device throughout the repo (#4982) * Remove unsupported targets in Device class (#4491) * Remove unsupported targets in Device class * Fixed broken tests * Deprecate UWP target * Allow to use UWP target on XAML * Fix build errors * comments * UWP == WinUI * Fix Build errors * Added more tests * fix * vbnm * More fixes * things Co-authored-by: Matthew Leibowitz <mattleibow@live.com> * [ci] Remove image override (#5053) * [ci] Remove image override * [ci] Don't provisioning windows Co-authored-by: Javier Suárez <javiersuarezruiz@hotmail.com> Co-authored-by: Matthew Leibowitz <mattleibow@live.com> Co-authored-by: Rui Marinho <me@ruimarinho.net>
Description of Change
Targets not supported by .NET MAUI like GTK, WPF, UWP have been marked as internal. WinUI has also been added as a target.
PR Checklist
Does this PR touch anything that might affect accessibility?