Hiding view before the pop animation completes is ugly about gtm-oauth2 HOT 7 CLOSED

Hiding the view immediately is important because the server typically displays 
some inappropriate instructions for the user once sign-in has completed.

I'm not sure which flow you are saying is ugly. Dismissal of the view 
controller without completing sign-in will not go through the popView method. 

Dismissal after signed-in is ugly.

When the user has signed in, the server will display an html message asking the 
user to copy an auth token. In the context of the sign-in controller, this 
messages does not make sense for the user, so the controller is trying to hide 
the message as quickly as possible.

Then I think we can let the web view to show a more user friendly message such 
as "You've Signed In Successfully", instead of hiding the whole view.

Try presenting and dismissing GTMOAuth2ViewControllerTouch as a model view, 
you'll see it is really ugly to hide the view before the dismissing animation 
completes.

We do not have control over what text the server displays in the html following 
sign-in.

No but you have absolute control over what the webview shows up.

@grobb...
> Hiding the view immediately is important because the server typically 
displays some inappropriate instructions for the user once sign-in has 
completed.

Yes WebView displays "please copy this code..."
But you may hide ONLY web view and show some kind of UIViewActivityIndicator

Also it will be useful because signIn process is not finished by that point
authCode obtained but it is not exchanged with token. Authentification process 
still continue end may fail.

But at this point view is hide instantly and also popViewBlock(); is called.
So i have ugly dismissed controller without animation (i present it moddaly)
And after that, in viewWillAppear of presentingViewController I can't use auth
because [auth canAutorize] == NO 


> I'm not sure which flow you are saying is ugly. Dismissal of the view 
controller without completing sign-in will not go through the popView method. 

Yes when i dissmiss it manually, but it goes when authCodeObtained than 
popViewBlock() is called
---
#1  in -[GTMOAuth2ViewControllerTouch popView] at 
GTMOAuth2ViewControllerTouch.m:405
#2  in -[GTMOAuth2ViewControllerTouch signIn:displayRequest:] at 
GTMOAuth2ViewControllerTouch.m:628
#3  in -[GTMOAuth2SignIn closeTheWindow] at GTMOAuth2SignIn.m:367
#4  in -[GTMOAuth2SignIn authCodeObtained] at GTMOAuth2SignIn.m:504
#5  in -[GTMOAuth2SignIn titleChanged:] at GTMOAuth2SignIn.m:458
#6  in -[GTMOAuth2ViewControllerTouch webViewDidFinishLoad:] at 
GTMOAuth2ViewControllerTouch.m:832
---


Also can you remove @private from GTMOAuth2ViewControllerTouch ivars
this will allow to fix a lot in subclasses