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

Update MMDrawerController.m #237

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jhjiang
Copy link

@jhjiang jhjiang commented Apr 8, 2014

Here is another way to support editing/dragging cells in the center view controller.
Make open pan gesture enable only at both sides, even at the edge of the container view.
MMDrawerSidePanGestureFlag & MMDrawerPanGestureSideContainerWidth are configurable to fit different situation.

Here is another way to support editing/dragging cells in the center view controller. 
Make open pan gesture enable only at both sides, even at the edge of the container view.
MMDrawerSidePanGestureFlag & MMDrawerPanGestureSideContainerWidth are configurable to fit different situation.
@kcharwood
Copy link
Contributor

@jhjiang Would you mind providing more information around the problem you are trying solve that isn't currently supported?

@jhjiang
Copy link
Author

jhjiang commented May 29, 2014

Please refer to below image.
It's allowed to limit users to drag the side containers out of either edges, only by starting "panning" from area A or B.
"MMDrawerPanGestureSideContainerWidth" is the width of area A or B.
"MMDrawerSidePanGestureFlag" is a switch to enable this function. Open it, and users can do any operations in the center area, which won't be catched by MMDrawerController.

1

BOOL sideContainedFlag = (_leftDrawerViewController != nil && CGRectContainsPoint(leftSideRect, point)) || (_rightDrawerViewController != nil && CGRectContainsPoint(rightSideRect, point));
return sideContainedFlag;
}
//-----Added by Jeffrey End-----

Choose a reason for hiding this comment

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

No need to add those lines as Git Blame will properly identify the author of any given line.

Copy link
Author

Choose a reason for hiding this comment

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

These comments are only used as individual signals in my local workspace.
Thanks for your kindly reminder all the same. :)

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