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

Wrap ActionSheet with a Modal #169

Merged
merged 7 commits into from
May 6, 2020

Conversation

BrodaNoel
Copy link
Contributor

@BrodaNoel BrodaNoel commented Apr 27, 2020

Issues fixed

#164

Video

https://youtu.be/4aJ8T-M6RR0

Texts/Examples

Yeap, added.

@bradbyte
Copy link
Collaborator

Hey @BrodaNoel, thanks for jumping on this -- I'm sorry I wasn't able to get something together sooner.

I think an initial pattern to a) get this out for your asap, and b) not risk any issues to existing usage would be to determine this with a useModal=true prop that is false by default. This would need to be added to the Android only props.

In the next breaking update this could be moved to the default.

Thoughts?

src/ActionSheet/index.tsx Show resolved Hide resolved
src/ActionSheet/index.tsx Outdated Show resolved Hide resolved
@BrodaNoel BrodaNoel requested a review from bradbyte April 28, 2020 18:06
BrodaNoel and others added 2 commits April 28, 2020 19:37
Co-Authored-By: Brad Bumbalough <bradbumbalough@users.noreply.github.com>
README.md Show resolved Hide resolved
@bradbyte bradbyte merged commit 9ed955d into expo:master May 6, 2020
bradbyte pushed a commit that referenced this pull request May 6, 2020
# [3.7.0](v3.6.0...v3.7.0) (2020-05-06)

### Features

* adds `useModal` prop wrap action sheet in a modal ([9ed955d](9ed955d)), closes [#169](#169)
@bradbyte
Copy link
Collaborator

bradbyte commented May 6, 2020

🎉 This PR is included in version 3.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@BrodaNoel BrodaNoel deleted the fix/164-hidden-on-android branch May 10, 2020 00:49
robertomartinez09515 added a commit to robertomartinez09515/action-sheet-react-best-proj that referenced this pull request Jul 25, 2022
# [3.7.0](expo/react-native-action-sheet@v3.6.0...v3.7.0) (2020-05-06)

### Features

* adds `useModal` prop wrap action sheet in a modal ([9ed955d](expo/react-native-action-sheet@9ed955d)), closes [#169](expo/react-native-action-sheet#169)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants