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

[iOS] Fix non visible TextColor issue in Entry using Visual #13051

Merged
merged 5 commits into from
Dec 19, 2020
Merged

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Dec 4, 2020

Description of Change

The main issue is solved in native Material Components iOS library:
https://github.com/material-components/material-components-ios/issues/10081

We did a color workaround #12246 to avoid the issue but we will investigate in release new binding for Material Components library. This PR just include a small change in the color workaround to fix the entry text no visible issue using the default color for now.

The default color alpha is -1, and that's why we set the default alpha to zero in the color workaround.

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

fix12809

Testing Procedure

Launch Core Gallery and navigate to the issue 12809 on iOS simulator or device. Write in each Entry and verify that the text color is always visible.

PR Checklist

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

@jsuarezruiz jsuarezruiz added the i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often label Dec 11, 2020
@rmarinho rmarinho added this to the 5.0.1 milestone Dec 16, 2020
@rmarinho rmarinho added this to In progress in v5.0.1 via automation Dec 16, 2020
@rmarinho rmarinho added this to To do in vNext+1 (5.0.0) via automation Dec 18, 2020
@rmarinho rmarinho removed this from In progress in v5.0.1 Dec 18, 2020
@rmarinho rmarinho modified the milestones: 5.0.1, 5.0.0 Dec 18, 2020
@rmarinho rmarinho moved this from To do to In Review in vNext+1 (5.0.0) Dec 18, 2020
@rmarinho
Copy link
Member

Failing tests not related

@rmarinho rmarinho merged commit ad4091a into 5.0.0 Dec 19, 2020
vNext+1 (5.0.0) automation moved this from In Review to Done Dec 19, 2020
@rmarinho rmarinho deleted the fix-12809 branch December 19, 2020 18:51
@samhouts samhouts added 5.0.0 Regression on 5.0.0 i/regression labels Jan 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
5.0.0 Regression on 5.0.0 a/entry a/visual i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often i/regression p/iOS 🍎 t/bug 🐛
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[Bug] Entry text is invisible on iOS
5 participants