Custom NavigationController Delegate about hero

[lkzhao]

You can probably do something like this instead of setting isHeroEnabled.

extension ModalNavigationController: UINavigationControllerDelegate {
    func navigationController(_ navigationController: UINavigationController,
                              willShow viewController: UIViewController,
                              animated: Bool) {
        //need to configure something on viewController.view
    }

    func navigationController(_ navigationController: UINavigationController, animationControllerFor operation: UINavigationControllerOperation, from fromVC: UIViewController, to toVC: UIViewController) -> UIViewControllerAnimatedTransitioning? {
        return Hero.shared.navigationController(navigationController, animationControllerFor:operation, from:fromVC)
    }
  
    func navigationController(_ navigationController: UINavigationController, interactionControllerFor animationController: UIViewControllerAnimatedTransitioning) -> UIViewControllerInteractiveTransitioning? {
        return Hero.shared.navigationController(navigationController, interactionControllerFor:animationController)
    }
}

[lkzhao]

Hero.shared is not avaliable on cocoapods and carthage yet. The current master contains a major API change. Will wait a few day before releasing the next version. In the mean time, you can use

pod "Hero", :git => 'https://github.com/lkzhao/Hero.git'

[felix-dumit]

I had to add nav.transitioningDelegate = Hero.shared instead of setting isHeroEnabled.
It turns out those two delegate methods are not being called at all. (even with setting isHeroEnabled) - is that expected?
EDIT: Actually it get's called once I push from the rootVC to the next one, that makes sense.

[lkzhao]

Oh. I guess I didn't understand your question clearly.

Yes, for the modal presentation you only have to set 'transitionDelegate'. Hero also sets the navigation controller's delegate for animating navigation controller's child view controllers. If you don't need that functionality then it is ok to just set the 'transitionDelegate'