Skip to content
This repository was archived by the owner on Jan 13, 2022. It is now read-only.

TTPostController: moves calls to super controller's view appear/disappear methods#442

Closed
echamberlain wants to merge 1 commit into
developmentfrom
unknown repository
Closed

TTPostController: moves calls to super controller's view appear/disappear methods#442
echamberlain wants to merge 1 commit into
developmentfrom
unknown repository

Conversation

@echamberlain

Copy link
Copy Markdown

Simplified replacement for pull request #436.

This pull request adds a call to the super view controller's viewDidDisappear: after the animation stops.

This request also changes when the super view controller's viewWillAppear: method is called, viewWillAppear: is now called before any animation starts and the super view controller's viewDidAppear: method is now called after the animation stops.

@vguerci

vguerci commented Feb 28, 2011

Copy link
Copy Markdown
Contributor

Currently testing it, replaced previous pull by this one, I'm submitting an app to Apple today with this...
...And working flawlessly so far! thanks Eric.

@jverkoey

jverkoey commented Mar 3, 2011

Copy link
Copy Markdown
Contributor

Tested and confirmed that this works. Can you squash your commits into one and force push back to this branch?

Also, for the commit message, please prefix it with [UI] for historical purposes.

Thanks for putting this together!

@echamberlain

Copy link
Copy Markdown
Author

Done.

@jverkoey

jverkoey commented Mar 4, 2011

Copy link
Copy Markdown
Contributor

And one last request for the commit message: we should try to make it more descriptive to the task. Something like [UI] TTPostController calls viewWillDisappear and viewDidDisappear on super controller when displayed. would be cool.

And then in the body of the commit, if you could add a simple description of the problem as it existed before this change and how this change fixes it, we'll be gold to merge this in.

Pardon the pickiness here, but this is one of those changes that might not be particularly obvious in the future if we don't better document the rationale.

… super controller when displayed.

This fix adds a call to the super view controller's viewDidDisappear: after the animation stops.  This commit also changes when the super view controller's viewWillAppear: method is called, viewWillAppear: is now called before any animation starts and the super view controller's viewDidAppear: method is now called after the animation stops.
@echamberlain

Copy link
Copy Markdown
Author

By your command.

@jverkoey

jverkoey commented Mar 4, 2011

Copy link
Copy Markdown
Contributor

Cool, thanks :)

@jverkoey

jverkoey commented Mar 4, 2011

Copy link
Copy Markdown
Contributor

Merge branch 'TTPostControllerFix' of https://github.com/echamberlain/three20 into development

Closed by 219417e.

diederich pushed a commit that referenced this pull request Apr 17, 2011
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants