On 11 Mar 2014, at 16:05, James Coleman <notifications@github.com> wrote:

> In SGBDrillDownController/SGBDrillDownController.h:
> 
> > @@ -46,6 +52,7 @@
> >  
> >  // Split behaviour
> >  - (void)replaceRightViewController:(UIViewController *)viewController animated:(BOOL)animated completion:(void(^)(void))completion;
> > +- (void)replaceRightViewController:(UIViewController *)viewController animated:(BOOL)animated animationType:(SGBDrillDownControllerAnimationType) animationType completion:(void (^)(void))completion;
> Ah! Good idea. If I implemented it that way, would it make sense to have the other methods available that are just convenience shortcuts to handle specific cases? Or is that making the API too broad?
> 

Probably, yeah.  I think it’s fine for the API to be just push/pop/replace and then set for people with more complex needs.

-- 
Simon