Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Android] Set TextColor to search icon in SearchBar #14766

Closed
wants to merge 2 commits into from
Closed

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

Set TextColor to search icon in SearchBar. In this way, is easy to manage the desire color. Until now this color depended on AndroidProject/Resources/values/styles.xml and the styles used. Without setting a TextColor this will be the fallback behavior.

@jfversluis What is your opinion in this case?

Issues Resolved

API Changes

None

Platforms Affected

  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

image

Testing Procedure

Launch Core Gallery and navigate to the issue 9266. If the Search icon color is Yellow, the test has passed.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@jfversluis
Copy link
Member

jfversluis commented Jul 11, 2022

Let's not do this for Forms anymore. While this doesn't introduce any new APIs, it still is a breaking change from a behavioral perspective. People who might've set the font color today expect the icon not to change. If we would merge this then suddenly their icons start to change without a way to revert that.

One way or the other we are going to have to introduce a breaking change for this which is something we would want to avoid. Sorry :(

Looking at the original issue we might already set the icon to the text color before Xamarin 4.4. I will need to dive in a bit deeper in that case.

@jfversluis
Copy link
Member

I think at this point it's safe to say this won't be happening for Xamarin.Forms anymore. Let's make it happen for .NET MAUI!

@jfversluis jfversluis closed this Jun 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Search icon color wrong with 4.4
2 participants