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

[Android] Entry Clear button invisible if BackgroundColor is a dark color #12391

Merged
merged 6 commits into from
Jul 6, 2021

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Oct 7, 2020

Description of Change

If the background property of an entry is updated to be a dark color, then the clear button isn't visible.
This PR include changes to set the ClearButton color using the TextColor property.

Issues Resolved

API Changes

None

Platforms Affected

  • Android
  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

fix12380

Testing Procedure

Launch Core Gallery and navigate to the issue 12380. If the ClearButton is visible in all cases, the test has passed.

PR Checklist

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

@jsuarezruiz
Copy link
Contributor Author

We have the same problem on iOS, added changes to tint the clear image icon.

@Transis-Felipe
Copy link
Contributor

Transis-Felipe commented Nov 2, 2020

@jsuarezruiz what do you think about change the underline entry color too?
Look that we can't see the entry control when it's not focused

@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Nov 2, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@PureWeen PureWeen removed this from In Review in vNext+1 (5.0.0) Nov 5, 2020
@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Nov 5, 2020
@samhouts samhouts removed this from In progress in v5.0.1 Nov 5, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@PureWeen PureWeen removed this from In Review in vNext+1 (5.0.0) Nov 5, 2020
@MrAndrewAu
Copy link

We have been suffering from this bug for a while. Greatly looking forward to it in the next Xamarin.Forms release!

@InquisitorJax
Copy link

@jsuarezruiz is there something blocking this guy from getting merged?

@Redth Redth added this to Issue In progress in 5.0.0 SR 4 (Planning) via automation Jun 3, 2021
@Redth Redth moved this from Issue In progress to PR Needs Review in 5.0.0 SR 4 (Planning) Jun 3, 2021
@Redth Redth requested a review from mattleibow June 28, 2021 15:12
v5.0.1 automation moved this from In progress to Reviewer approved Jul 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Development

Successfully merging this pull request may close these issues.

[Bug] Android: Entry Clear button becomes invisible when changing Entry.BackgroundColor to dark color
7 participants