-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Match Catalyst Template to iOS for Shell Flyout Items #25056
Conversation
@@ -537,6 +537,16 @@ internal static DataTemplate CreateDefaultFlyoutItemCell(string textBinding, str | |||
return grid; | |||
}); | |||
} | |||
|
|||
#if NETSTANDARD | |||
sealed class OperatingSystem |
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.
Is this the right place for it? Do we need this in more places?
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.
Maybe? The problem I ran into putting this into a more general place is that you start to get errors about "OperatingSystem" is defined in multiple places. Unfortunately, NS has the System.OperatingSystem
type it just doesn't have anything useful for us.
Another thought I had was to just call it OperatingSystemNS
and then add a special using at the top, which did work.
Not sure the cleverest way to make this reusable.
Maybe we add our own internal static? MAUIOS
that we can use generally?
/backport to release/9.0.1xx |
Started backporting to release/9.0.1xx: https://github.com/dotnet/maui/actions/runs/11335293225 |
* Match Catalyst Template to iOS * - NS Fixes
Description of Change
When we added
Catalyst
during our move from XF to MAUI we forgot to include certain styles forCatalyst
alongsideiOS
Fixes
Before
After