EDIT: These problems appear to be fixed as of iOS 7.1 / Xcode 5.1.1. (Possibly earlier, as I haven't been able to test all versions. Definitely after iOS 7.0, since I tested that one.) When you create a popover segue from a UIBarButtonItem
, the segue makes sure that tapping the popover again hides the popover rather than showing a duplicate. It works right for the new UIPresentationController
-based popover segues that Xcode 6 creates for iOS 8, too.
Since my solution may be of historical interest to those still supporting earlier iOS versions, I've left it below.
If you store a reference to the segue's popover controller, dismissing it before setting it to a new value on repeat invocations of prepareForSegue:sender:
, all you avoid is the problem of getting multiple stacking popovers on repeated presses of the button -- you still can't use the button to dismiss the popover as the HIG recommends (and as seen in Apple's apps, etc.)
You can take advantage of ARC zeroing weak references for a simple solution, though:
1: Segue from the button
As of iOS 5, you couldn't make this work with a segue from a UIBarButtonItem
, but you can on iOS 6 and later. (On iOS 5, you'd have to segue from the view controller itself, then have the button's action call performSegueWithIdentifier:
after checking for the popover.)
2: Use a reference to the popover in -shouldPerformSegue...
@interface ViewController
@property (weak) UIPopoverController *myPopover;
@end
@implementation ViewController
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender {
// if you have multiple segues, check segue.identifier
self.myPopover = [(UIStoryboardPopoverSegue *)segue popoverController];
}
- (BOOL)shouldPerformSegueWithIdentifier:(NSString *)identifier sender:(id)sender {
if (self.myPopover) {
[self.myPopover dismissPopoverAnimated:YES];
return NO;
} else {
return YES;
}
}
@end
3: There's no step three!
The nice thing about using a zeroing weak reference here is that once the popover controller is dismissed -- whether programmatically in shouldPerformSegueWithIdentifier:
, or automatically by the user tapping somewhere else outside the popover -- the ivar goes to nil
again, so we're back to our initial state.
Without zeroing weak references, we'd have to also:
- set
myPopover = nil
when dismissing it in shouldPerformSegueWithIdentifier:
, and
- set ourself as the popover controller's delegate in order to catch
popoverControllerDidDismissPopover:
and also set myPopover = nil
there (so we catch when the popover is automatically dismissed).
Just call dismiss method using parent's presentedViewController property, like ....
self.presentedViewController.dismissViewControllerAnimated(true, completion: nil)
For Swift 3.0
self.presentedViewController?.dismiss(animated: true, completion: nil)
Best Answer
I take no credit for this since I got every bit of it from going through multiple StackOverflow threads, but I got this working with:
In my storyboard I control-dragged from my settings bar button item to
MyViewController
and connected it to thetoggleSettingsInPopover
action. Then I control-dragged fromMyViewController
to the view for the settings to create the segue, set its type topopover
, set its identifier toSettings
, set its directions to up and left (the toolbar is at the bottom of the screen and the button is at the right end), then dragged from itsAnchor
to the bar button item I connected to the action.