-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add react strict dom #59
Conversation
@@ -0,0 +1,50 @@ | |||
# React + TypeScript + Vite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At some point we will probably want to make it more personalized to the project but nbd for now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, def. We should probably just start adding these things as backlog items to the project. I'll add one for Tackle Box README
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we think we will want to bring back storybook as we get things setup?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything look good! Only real question right now is if we need to keep anything around in the src
file or curious what your thoughts are around previewing components as we work on them.
@cgjohn R.e. src, let's keep it for now cause it might help us debug the web build when we set up Storybook. R.e. previewing, we can add them to the iOS app for now in the ui.tsx route. Def should come up with a better plan soon but I'll just merge this as is and we can come up with a better solution as we build out the components. |
Swaps out React Native Web for
react-strict-dom
and adds a simple button example