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

Add FloatingComponent prop #162

Merged
merged 13 commits into from
Mar 29, 2020
Merged

Add FloatingComponent prop #162

merged 13 commits into from
Mar 29, 2020

Conversation

vforvasile
Copy link
Contributor

Hey!

I added the new prop, updated Props Docs, Readme preview, types and examples. Also merged with latest changes from rc15. (Tested examples with react-navigation and rn-native-navigation, couldn't run expo example but I'm certain it should work)

Here's the original recorded preview:
https://imgur.com/a/w8CNEgw
in case the gif below is not OK

Copy link
Owner

@jeremybarbet jeremybarbet 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, it looks good and I'm totally good with the introduction of this props.

Even though I'm kinda asking changes that will make you remove the examples you did and change the props name. Sorry about that!

src/options.ts Outdated Show resolved Hide resolved
src/index.tsx Outdated Show resolved Hide resolved
docs/PROPSMETHODS.md Outdated Show resolved Hide resolved
examples/expo/App.js Outdated Show resolved Hide resolved
@vforvasile vforvasile changed the title Add menuOverlay prop Add FloatingComponent prop Mar 29, 2020
Copy link
Contributor Author

@vforvasile vforvasile left a comment

Choose a reason for hiding this comment

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

I pushed everything to my branch, @jeremybarbet can you take a look again?

Copy link
Owner

@jeremybarbet jeremybarbet left a comment

Choose a reason for hiding this comment

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

Few minor changes and we are good to merge! Thanks

src/index.tsx Outdated Show resolved Hide resolved
src/options.ts Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/PROPSMETHODS.md Outdated Show resolved Hide resolved
docs/PROPSMETHODS.md Outdated Show resolved Hide resolved
src/options.ts Outdated Show resolved Hide resolved
Copy link
Contributor Author

@vforvasile vforvasile left a comment

Choose a reason for hiding this comment

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

all good?

@jeremybarbet
Copy link
Owner

Sweet, it looks good! Thank you for quick response to that!

@jeremybarbet jeremybarbet merged commit 269934e into jeremybarbet:master Mar 29, 2020
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.

2 participants