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

Disable web-preview for TextStyleProps example #2656

Merged
merged 2 commits into from
Jun 24, 2021

Conversation

IjzerenHein
Copy link
Contributor

@IjzerenHein IjzerenHein commented Jun 24, 2021

Why

react-native-web does not support the deprecated Slider component causing the TextStyleProps example to fail with a React minified error upon loading the page.

As discussed here:
#2476 (comment)
@Simek

before

image

after

image

How

  • Make example only available on ios and android

Why

  • See screenshot
  • Ran yarn prettier to verify the file doesn't contain any styling issues

`react-native-web` does not support the deprecated `Slider` component causing the example to fail with a React minified error upon loading the page.
@netlify
Copy link

netlify bot commented Jun 24, 2021

✔️ Deploy Preview for react-native ready!

🔨 Explore the source changes: 00c007c

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

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

Copy link
Collaborator

@Simek Simek left a comment

Choose a reason for hiding this comment

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

Hi @IjzerenHein, thank you for catching this issue out and pushing the fix! 🙂

@IjzerenHein
Copy link
Contributor Author

IjzerenHein commented Jun 24, 2021

Hi @IjzerenHein, thank you for catching this issue out and pushing the fix! 🙂

Your welcome, LGTM :D

@Simek Simek merged commit 191010e into facebook:master Jun 24, 2021
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.

3 participants