Skip to content
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

Fixed the picker title's color #23075

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fixed the picker title's color #23075

wants to merge 3 commits into from

Conversation

kubaflo
Copy link
Contributor

@kubaflo kubaflo commented Jun 15, 2024

Issues Fixed

Fixes #16737

@kubaflo kubaflo requested a review from a team as a code owner June 15, 2024 12:19
@kubaflo kubaflo changed the title Fixed the picker title's color (#16737) Fixed the picker title's color Jun 15, 2024
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jun 15, 2024
Copy link
Contributor

Hey there @kubaflo! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@Redth
Copy link
Member

Redth commented Jun 18, 2024

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Comment on lines 17 to +20
if (PlatformInterop.CreateEditTextColorStateList(platformPicker.TextColors, titleColor.ToPlatform()) is ColorStateList c)
platformPicker.SetHintTextColor(c);
else if(picker.TextColor == picker.TitleColor)
platformPicker.SetHintTextColor(titleColor.ToPlatform());
Copy link
Member

Choose a reason for hiding this comment

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

Why do we have this color state list instead of just always setting the color?

Copy link
Contributor Author

@kubaflo kubaflo Jul 19, 2024

Choose a reason for hiding this comment

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

I don't know. Maybe it should be removed... However, I'm not sure of the potential implications removing it might have

@dartasen
Copy link
Contributor

@mattleibow Any news for this one ? 😄

@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Title colour on Android Picker, initially appears grey.
7 participants