-
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
[Android] Custom shell back button color #22940
base: main
Are you sure you want to change the base?
Conversation
@@ -392,7 +394,7 @@ protected virtual async void UpdateLeftBarButtonItem(Context context, AToolbar t | |||
DrawerArrowDrawable icon = null; | |||
bool defaultDrawerArrowDrawable = false; | |||
|
|||
var tintColor = Colors.White; | |||
var tintColor = Shell.GetForegroundColor(page) ?? Colors.White; |
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.
Does this make it match the behavior on iOS and Android?
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.
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 the icon misaligned on iOS?
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.
I used the default icon that has been included in the test app. It is too big, so it is why it looks like this. It's a different bug but not related to this change
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
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 the icon misaligned on iOS?
Would this fix an SVG that has multiple colors as well? like if you wanted a company logo with multiple colors because at the moment those are all white as well. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Commenter does not have sufficient privileges for PR 22940 in repo dotnet/maui |
Issues Fixed
Fixes #22450