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

AHKNavigationController #5669

Closed
wants to merge 9 commits into from
Closed

Conversation

yamill
Copy link
Contributor

@yamill yamill commented Feb 1, 2016

This enables NavigatorIOS swipe-back when Navigation Bar is hidden.
Issue: #1341

See: https://github.com/fastred/AHKNavigationController for more detail

yamill and others added 2 commits February 1, 2016 11:09
Implemented this so that it re-enables the interactive pop gesture when
the navigation bar is hidden or a custom back button is used.
@facebook-github-bot
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @sahrens, @dougbankspersonal and @nicklockwood to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Feb 1, 2016
@insraq
Copy link

insraq commented Feb 17, 2016

I've manually patched this with RN 0.20. Works perfectly. Can we have this merged into the core? Thanks!

@insraq
Copy link

insraq commented Feb 17, 2016

Actually you have to add those files to the xcode project file.

@yamill
Copy link
Contributor Author

yamill commented Feb 17, 2016

@insraq thanks for catching that! 👍 i just added it to the xcode project. hopefully this will get merged into the core.

@facebook-github-bot
Copy link
Contributor

@yamill updated the pull request.

@facebook-github-bot
Copy link
Contributor

@yamill updated the pull request.

@satya164
Copy link
Contributor

cc @nicklockwood

@facebook-github-bot
Copy link
Contributor

@yamill updated the pull request.

@mkonicek
Copy link
Contributor

This is a large PR and the NavigatorIOS is not maintained by Facebook so I don't think anyone has bandwidth to review this.

Posted in this read-only group, feel free to follow along there:
https://www.facebook.com/groups/reactnativeoss/permalink/1533536423609653

@mkonicek
Copy link
Contributor

mkonicek commented Apr 1, 2016

Actually I'll close this since the NavigatorIOS is not maintained. Feel free to copy the NavigatorIOS code into your project and modify it there, or fork it on your GitHub account.

@mkonicek mkonicek closed this Apr 1, 2016
@mkonicek
Copy link
Contributor

mkonicek commented Apr 1, 2016

Thanks for understanding!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants