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

View-only mode works in react-native (with installed or shimmed react-dom) #368

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Shaddix
Copy link
Contributor

@Shaddix Shaddix commented Jan 29, 2022

Fixes #363.
This is actually the only change that is needed for a library to work in react-native.
(secure id generation is not supported in react-native).

In react-native project there's still one thing needs to be done:

  • either npm install react-dom (though it's never actually called unless you try to use Designer with RenderIndicator).
  • shim react-dom in react-native app using e.g. technique from the link

(if it makes sense, these instructions could be added to README.MD)

@Shaddix Shaddix requested a review from prevwong as a code owner January 29, 2022 17:03
@Shaddix Shaddix changed the title View-only mode works in react-native (with installed or shimmed react-dom) View-only mode works in react-native (with installed or shimmed react-dom), fixes #363 Jan 29, 2022
@Shaddix Shaddix changed the title View-only mode works in react-native (with installed or shimmed react-dom), fixes #363 View-only mode works in react-native (with installed or shimmed react-dom) Jan 29, 2022
@netlify
Copy link

netlify bot commented Jan 29, 2022

✔️ Deploy Preview for craftjs ready!

🔨 Explore the source changes: 32e0d5c

🔍 Inspect the deploy log: https://app.netlify.com/sites/craftjs/deploys/61f5734e3dc92100079f69dd

😎 Browse the preview: https://deploy-preview-368--craftjs.netlify.app

@prevwong prevwong force-pushed the develop branch 2 times, most recently from f09aac2 to d8ad5cf Compare March 16, 2022 19:00
@prevwong prevwong force-pushed the develop branch 8 times, most recently from af39d6c to 5f5223d Compare April 19, 2022 12:25
@prevwong prevwong force-pushed the main branch 3 times, most recently from f14e620 to 84adfd1 Compare October 6, 2023 08:34
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.

Support react-native (view-only)
1 participant