UINavigationController inside a UITabBarController inside a UISplitViewController presented modally on iPhone

The problem with Peter's solution is that it will fall apart with the iPhone 6 +. How so? With that code, if an iPhone 6 + is in portrait orientation - the detail view pushes onto the navigation stack. All is well, so far. Now, rotate into landscape, and then you'll have the detail view showing as the detail view and the master view.

You'll need the split view controller's delegate to implement two methods:

- (BOOL)splitViewController:(UISplitViewController *)splitViewController showDetailViewController:(UIViewController *)detailVC sender:(id)sender
{
    UITabBarController *masterVC = splitViewController.viewControllers[0];

    if (splitViewController.traitCollection.horizontalSizeClass == UIUserInterfaceSizeClassCompact)
        [masterVC.selectedViewController showViewController:detailVC sender:sender];
    else
        [splitViewController setViewControllers:@[masterVC, detailVC]];

    return YES;
}

And now, you'll need to return the top view controller from the selected tab's navigation controller:

- (UIViewController*)splitViewController:(UISplitViewController *)splitViewController separateSecondaryViewControllerFromPrimaryViewController:(UIViewController *)primaryViewController
{
    UITabBarController *masterVC = splitViewController.viewControllers[0];

    if ([(UINavigationController*)masterVC.selectedViewController viewControllers].count > 1)
        return [(UINavigationController*)masterVC.selectedViewController popViewControllerAnimated:NO];
    else
        return nil; // Use the default implementation
}

With this solution, everything pushes onto the navigation stack when it should and also updates the detail view correctly on the iPad/6+ landscape.


I figured out how to put the detail on to the master's UINavigationController instead of presenting it modally over the UITabBarController.

Using the UISplitViewControllerDelegate method

- splitViewController:showDetailViewController:sender:

In case the UISplitViewController is collapsed get the masters navigation controller and push the detail view onto this navigation controller:

- (BOOL)splitViewController:(UISplitViewController *)splitViewController
   showDetailViewController:(UIViewController *)vc
                     sender:(id)sender {
    NSLog(@"UISplitViewController collapsed: %d", splitViewController.collapsed);

    // TODO: add introspection
    if (splitViewController.collapsed) {
        UITabBarController *master = (UITabBarController *) splitViewController.viewControllers[0];
        UINavigationController *masterNavigationController = (UINavigationController *)master.selectedViewController;

        // push detail view on the navigation controller
        //[masterNavigationController pushViewController:vc animated:YES];
        // push was not always working (see discussion in answer below), use showViewController instead
        [masterNavigationController showViewController:vc sender:sender];

        return YES;
    }

    return NO;
}

The answer of @PeterOettl to his own question put me on the right way and is great for that. So the credit belongs to him.

I have nearly the same storyboard structure as him, but as vc is a navigationController I get a runtime error saying

'Pushing a navigation controller is not supported'

As said, that is because vc is the navigationController of the detail view and not the viewController of the detail view.

Note that I am surprised that @PeterOettl does not get that error in his case also, as the segue given in the storyboard picture, points to the navigation controller of the detail view.

Therefore the code should like that (in Swift) simply adding

let detailViewControllerNavigationController = (vc as UINavigationController).viewControllers[0] as UIViewController

and pushing detailViewControllerNavigationController instead of vc

and the whole code is

func splitViewController(splitViewController: UISplitViewController, showDetailViewController vc: UIViewController, sender: AnyObject?) -> Bool {
    println("UISplitViewController collapsed: \(splitViewController.collapsed)")
    if (splitViewController.collapsed) {
        let master = splitViewController.viewControllers[0] as UITabBarController
        let masterNavigationController = master.selectedViewController as UINavigationController

        let detailViewControllerNavigationController = (vc as UINavigationController).viewControllers[0] as UIViewController

        masterNavigationController.pushViewController(detailViewControllerNavigationController, animated: true)

        return true
    } else {
        return false
    }
}

Also note that this code is put in the AppDelegate.swift of the master-detail example of Xcode where a tab bar is added in the master view.

EDIT

In the comments we discussed with @PeterOettl of the difference between .pushViewController and .showViewController.

Apple documentation says :

showViewController:sender:

This method pushes a new view controller onto the navigation stack in a similar way as the pushViewController:animated: method. You can call this method directly if you want but typically this method is called from elsewhere in the view controller hierarchy when a new view controller needs to be shown.

Available in iOS 8.0 and later.