Skip to content

Allow select scheme to change color scheme manually #1430

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

Merged
merged 5 commits into from
Jul 29, 2021

Conversation

s123121
Copy link
Contributor

@s123121 s123121 commented Jul 26, 2021

Description

Currently, this lib can change dark/light mode base on system setting. However, most of the time, user want to change the theme directly on the app. This PR allow app developer to change theme directly without depending on user's appearance preferences

Also fix a bug when app go to background, Apperance module listen to the wrong color scheme

Changelog

Add selectScheme to change the color scheme manually

@s123121
Copy link
Contributor Author

s123121 commented Jul 26, 2021

I have change per requested and also fix a bug regarding Apperance
More info here: facebook/react-native#28525
Also currently, on Android, developer need to change the java code so that Appearance can listen to the colorScheme change (maybe add the warning inside doc?)
facebook/react-native#28823

@s123121 s123121 changed the title Add set scheme method Allow select scheme to change color scheme manually Jul 26, 2021
Copy link
Collaborator

@ethanshar ethanshar left a comment

Choose a reason for hiding this comment

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

@s123121 Looks great!
Thanks for contributing (:

@ethanshar ethanshar merged commit 14f43f0 into wix:master Jul 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants