Skip to content

add target/source ViewController in STPopupControllerTransitioningCon…#98

Open
calvinsug wants to merge 3 commits into
kevin-lyn:masterfrom
calvinsug:master
Open

add target/source ViewController in STPopupControllerTransitioningCon…#98
calvinsug wants to merge 3 commits into
kevin-lyn:masterfrom
calvinsug:master

Conversation

@calvinsug

Copy link
Copy Markdown

add target/source View Controller for giving access to TargetViewController when using Custom Transition style popup


@param animated YES if animated
*/
- (void)popToRootViewControllerAnimated:(BOOL)animated;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is this method removed?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

oops, i didn't remember i deleted this method.

UIViewController *topViewController = self.topViewController;

STPopupControllerTransitioningContext *context = [self convertTransitioningContext:transitionContext];

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please revert unneeded change if it's possible.

*/
- (void)popViewControllerAnimated:(BOOL)animated;


Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please revert unneeded change if it's possible.


/**
Pops all view controllers from the stack until it reaches the root view controller

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please revert unneeded change if it's possible.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

ok, done

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