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

Added the ability to prevent pan to close within the drawer. #87

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

colincameron
Copy link

Panning to close was interfering with the ability to swipe to delete items in a table view within the drawer view controller, so I have added a property to prevent pan gestures being recognised if they are within the drawer.

This is similar to #76, but still allows panning to close outside the drawer.

Allow pan gestures to optionally pass through to the drawerViewController.

Prevents e.g. swipe-to-delete on table view cells from being over-ridden to close the drawer.
Necessary to allow panToCloseInsideDrawerEnabled property to appear in Interface Builder.
@timusus timusus requested a review from ykyouhei May 24, 2018 06:44
@timusus
Copy link
Collaborator

timusus commented May 24, 2018

@colincameron does this supersede #76? In other words, if we merge this in can we close #76?

@colincameron
Copy link
Author

@timusus yes as far as I can remember this solves the same issue with some improvements.

@ccosnean
Copy link

ccosnean commented Aug 2, 2018

please merge this...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants