Skip to content
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

Minor wording changes in PlatformColor.md #2053

Merged
merged 1 commit into from
Jul 24, 2020

Conversation

eduardopelitti
Copy link
Contributor

I found this section specially hard to read, so I'm proposing some minor tweaks that I think will make it easier to understand.

I also added a mention to the fallback values which the function can receive, taking from the original PR which introduced the changes (facebook/react-native#27908)

I don't know what the exact policy is on this, but since this was introduced in RN 0.63.0, perhaps this could also be clarified on the docs, as using it from an older version will throw a _reactNative.PlatformColor is not a function error.

I found this section specially hard to read, so I'm proposing some minor tweaks that I think will make it easier to understand. 

I also added a mention to the fallback values which the function can receive, taking from the original PR which introduced the changes (facebook/react-native#27908)

I don't know what the exact policy is on this, but since this was introduced in RN 0.63.0, perhaps this could also be clarified on the docs, as using it from an older version will throw a `_reactNative.PlatformColor is not a function` error.
@react-native-bot
Copy link

Deploy preview for react-native ready!

Built with commit c3fc0be

https://deploy-preview-2053--react-native.netlify.app

Changes to docs/ are reflected in the next "master" version.

Thank you for your contributions.

How to ContributeDocumentation Sources

@Simek Simek requested a review from rachelnabors July 19, 2020 09:46
Copy link
Contributor

@rachelnabors rachelnabors left a comment

Choose a reason for hiding this comment

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

This is clearer. Thank you :)

@rachelnabors rachelnabors merged commit 39a648a into facebook:master Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants