-
-
Notifications
You must be signed in to change notification settings - Fork 110
dannyhw/fix/nwui template issues #550
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
Conversation
fix: update more dependencies fix: keep reanimted v3 for nativewind fix: keep reanimted v3 for nativewind fix fix: ensure we install 53 until we are ready to support 54 fix: tests should run fix: extra changeset fix: use reanimated 4 always run tests with node fix: ensure we install 53 until we are ready to support 54 chore(release): version packages fix: generating projects with yarn chore(release): version packages fix: tests fix fix: snapshots fix: extra comma breaking chore(release): version packages
fix: resolve TypeScript error in authentication select options fix: scripts
| screenOptions={{ | ||
| tabBarActiveTintColor: 'black', | ||
| }}> | ||
| <Tabs> |
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.
this was causing the icon to be barely visible in dark mode
| onPress={() => bottomSheetModalRef.current?.present()} | ||
| /> | ||
| <Sheet ref={bottomSheetModalRef} snapPoints={[200]}> | ||
| <Sheet ref={bottomSheetModalRef} snapPoints={[200]} enableDynamicSizing={false}> |
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.
fixes bottom sheet not opening
01bc0c1 to
ede6be8
Compare
fix: tsc error fix: changeset
ede6be8 to
11b6e68
Compare
|
@dannyhw can you go through this PR and let me know what if anything needs to be updated given my recent updates to NWUI? If you think it's easier, we can close this for now and open a new PR for any desired changes. |
|
@danstepanov I think enough changes have probably been made that its easier to close this |
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):