Skip to content

support high contrast colors - #2640

Merged
Simek merged 3 commits into
react:masterfrom
birkir:patch-1
Jun 22, 2021
Merged

support high contrast colors#2640
Simek merged 3 commits into
react:masterfrom
birkir:patch-1

Conversation

@birkir

@birkirbirkir commented Jun 5, 2021

Copy link
Copy Markdown
Contributor

Update docs to support High Contrast colors, outlined in this PR: react/react-native#31651

@netlify

netlifyBot commented Jun 5, 2021

Copy link
Copy Markdown

✔️ Deploy Preview for react-native ready!

🔨 Explore the source changes: 4a39eca

🔍 Inspect the deploy log: https://app.netlify.com/sites/react-native/deploys/60d1b852d1f04200077370b1

😎 Browse the preview: https://deploy-preview-2640--react-native.netlify.app

@SimekSimek added 🕐 Waiting on PR These may be waiting for a PR to the facebook/react-native to be merged Wait on future Release This indicates a PR that updates the doc to match a future release. labels Jun 5, 2021
@birkir

Copy link
Copy Markdown
ContributorAuthor

react/react-native@4b9d9dd
v65 - react-native-community/releases#235

Comment threaddocs/dynamiccolorios.md Outdated
@SimekSimek removed 🕐 Waiting on PR These may be waiting for a PR to the facebook/react-native to be merged Wait on future Release This indicates a PR that updates the doc to match a future release. labels Jun 21, 2021

@SimekSimek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As you mentioned the feature has landed in 0.65 branch, so this PR is ready to merge to next version now. 🙂

Thank you for posting update to the docs, and working on high contrast color support in the first place. Please fix the comments in the first code block, besides that changes LGTM, great work! 👍

Co-authored-by: Bartosz Kaszubowski <gosimek@gmail.com>
@birkir

Copy link
Copy Markdown
ContributorAuthor

Accepted, Thanks @Simek

@Simek
Simek merged commit ac6f9a5 into react:masterJun 22, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@birkir@Simek@facebook-github-bot