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

feat: add Windows and macOS support #8570

Merged
merged 5 commits into from Jul 28, 2020

Conversation

kaiguo
Copy link
Contributor

@kaiguo kaiguo commented Jul 10, 2020

react-native-gesture-handler doesn't have Windows/macOS implementation yet, use the web version as stubs for now to enable navigation features for Windows and mac.

Close microsoft/react-native-windows#3884
Close microsoft/react-native-windows#4044

@kaiguo kaiguo changed the title Add Windows support feat: add Windows support Jul 10, 2020
@kaiguo kaiguo marked this pull request as ready for review July 14, 2020 18:10
@elcssmouhsine
Copy link

could you also add react-native-macos support while at it :

animationEnabled: Platform.OS !== 'web' && Platform.OS !== 'windows' && Platform.OS !== 'macos',

@kaiguo
Copy link
Contributor Author

kaiguo commented Jul 15, 2020

could you also add react-native-macos support while at it

Yep, added : )

@kaiguo kaiguo changed the title feat: add Windows support feat: add Windows and macOS support Jul 15, 2020
@elcssmouhsine
Copy link

react-native-gesture-handler does not support macos

issue #1052

Copy link
Member

@satya164 satya164 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. I have a couple of minor changes. We can merge after the changes.

packages/drawer/src/views/Drawer.tsx Outdated Show resolved Hide resolved
packages/drawer/src/views/GestureHandlerCompat.tsx Outdated Show resolved Hide resolved
packages/stack/src/views/GestureHandlerCompat.tsx Outdated Show resolved Hide resolved
packages/stack/src/views/MaskedView.macos.tsx Outdated Show resolved Hide resolved
@kaiguo
Copy link
Contributor Author

kaiguo commented Jul 28, 2020

Thanks @satya164! Just pushed out another commit to address your feedbacks.

Copy link
Member

@satya164 satya164 left a comment

Choose a reason for hiding this comment

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

Thanks!

@satya164 satya164 merged commit 8468c46 into react-navigation:main Jul 28, 2020
@kaiguo kaiguo deleted the add-windows-support branch July 28, 2020 22:15
@kaiguo
Copy link
Contributor Author

kaiguo commented Jul 30, 2020

Hey @satya164 do we know when the next release will be? Looks like the current release doesn't have this change and your refactor yet.

@satya164
Copy link
Member

I need to finish some other stuff before doing a release

@Zorgatone
Copy link

Waiting for the release as well

@lh9403
Copy link

lh9403 commented Apr 7, 2021

Is MacOS supported now?

joshuapinter pushed a commit to cntral/react-navigation that referenced this pull request Sep 29, 2021
react-native-gesture-handler doesn't have Windows/macOS implementation yet, use the web version as stubs for now to enable navigation features for Windows and mac.

Close microsoft/react-native-windows#3884
Close microsoft/react-native-windows#4044
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.

react-navigation-stack causes errors when imported Support for React-navigation
5 participants