Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix switch trackColor on Android. fixes #23962 #23977

Closed

Conversation

dulmandakh
Copy link
Contributor

Summary

fixes #23962, where trackColor is reset when value changed. This PR will set trackColor corresponding trackColor every-time value changes.

Changelog

[Android] [Changed] - Fix Switch trackColor

Test Plan

CI is green, and added example to RNTester app.

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. Partner labels Mar 16, 2019
Copy link

@analysis-bot analysis-bot left a comment

Choose a reason for hiding this comment

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

Code analysis results:

  • eslint found some issues. Run yarn lint --fix to automatically fix problems.

RNTester/js/SwitchExample.js Outdated Show resolved Hide resolved
RNTester/js/SwitchExample.js Outdated Show resolved Hide resolved
RNTester/js/SwitchExample.js Outdated Show resolved Hide resolved
RNTester/js/SwitchExample.js Outdated Show resolved Hide resolved
@dulmandakh dulmandakh changed the title fix switch track color on Android. fixes #23962 fix switch trackColor on Android. fixes #23962 Mar 16, 2019
@dulmandakh dulmandakh requested a review from cpojer March 16, 2019 13:00
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @dulmandakh in d6ee448.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Mar 16, 2019
facebook-github-bot pushed a commit that referenced this pull request Mar 18, 2019
Summary:
In #23977 , it changed switch example, it leads snapshots test failed.

cc hramos .

[iOS] [Fixed] - Fixed test_ios Switch test failed
Pull Request resolved: #24009

Differential Revision: D14504806

Pulled By: cpojer

fbshipit-source-id: 0c47226a3da0cba2371c627f42f62835b3aac810
grabbou pushed a commit that referenced this pull request Mar 22, 2019
Summary:
fixes #23962, where trackColor is reset when value changed. This PR will set trackColor corresponding trackColor every-time value changes.

[Android] [Changed] - Fix Switch trackColor
Pull Request resolved: #23977

Differential Revision: D14495206

Pulled By: hramos

fbshipit-source-id: d712f540cd3f8359d6e85f79c12732689870a112
grabbou pushed a commit that referenced this pull request Mar 25, 2019
Summary:
In #23977 , it changed switch example, it leads snapshots test failed.

cc hramos .

[iOS] [Fixed] - Fixed test_ios Switch test failed
Pull Request resolved: #24009

Differential Revision: D14504806

Pulled By: cpojer

fbshipit-source-id: 0c47226a3da0cba2371c627f42f62835b3aac810
@dulmandakh dulmandakh deleted the android-switch-trackcolor branch October 15, 2019 05:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Component: Switch Contributor A React Native contributor. Merged This PR has been merged. Platform: Android Android applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

trackColor.false doesn't persist when Android Switch component is toggled
5 participants