You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
STPopupController is supposed to work just like a UINavigationController. Currently, it only supports pushViewController: animated: completion: and popViewControllerAnimated:.
I'd like to propose the following UINavigationController functionality be implemented:
@AnthonyMDev Thanks for the suggestions.
The reason I haven't supported all the navigation controller functions is I want to keep it simple and most of the time we don't need them. And I think we will only push 2-3 view controllers into popupController? Having a complex view controllers stack in a popup may not be a good UX. But it is still nice to have popToViewController:animated and popToRootViewControllerAnimated.
The UIToolbar functionality would be very usable though.
@kevin0571 That's fair. I've updated the proposal.
I still think the ability to access the viewControllers property would be very valuable and easy to implement, as it's already there, just private. Making it editable may be too complex.
The rest of these you would be open to implementation on though?
I'd be interested in helping to implement some of this.
STPopupController
is supposed to work just like aUINavigationController
. Currently, it only supportspushViewController: animated: completion:
andpopViewControllerAnimated:
.I'd like to propose the following
UINavigationController
functionality be implemented:topViewController
(Implemented in PR Make 'topViewController' method public #43)popViewControllerAnimated
should return the popped controllerpopToViewController: animated
popToRootViewControllerAnimated
viewControllers
setViewControllers: animated:
- (Rejected)UIToolbar
functionalityUIViewController
'stoolbarItems
toolbarHidden
propertysetToolbarHidden: animated:
delegate
that has methods mirroring `UINavigationControllerDelegateI'd be happy to assist in implementing some of these features, but I wanted to open up discussion on them first.
The text was updated successfully, but these errors were encountered: