Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-7773]iOS: Back/close animation broken when a NavigationGroup is in a TabGroup #2295

Merged
merged 7 commits into from
May 31, 2012
Merged

[TIMOB-7773]iOS: Back/close animation broken when a NavigationGroup is in a TabGroup #2295

merged 7 commits into from
May 31, 2012

Conversation

vishalduggal
Copy link
Contributor

Test is in JIRA

@@ -1162,6 +1162,31 @@ -(void)viewDidDetach
// for subclasses
}

-(void)parentWillAppear:(id)args
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like these methods should not be triggered until after the current view appears/disappears, or there could be issues where a view thinks its parent is visible before it's on screen. Probably should behave like TiUIWindow below.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These methods are tied to the ViewControllers view(Will/Did)(Appear/Disappear) calls. They are always initiated from the TiUIWindow class.

@sptramer
Copy link
Contributor

@vishalduggal Code reviewed, please address comments.

@vishalduggal
Copy link
Contributor Author

Encapsulated the state propagation in a viewAttached check. Ready for re-review

@sptramer
Copy link
Contributor

Code reviewed, APPROVED. Not quite sure about whether or not the view attachment is required after all, but it seems like a good safety check since views are accessed during the parent visible chain.

@sptramer
Copy link
Contributor

Tested, iPhone Sim 5.1. Light KS smoke and JIRA test. Passed, APPROVED.

sptramer pushed a commit that referenced this pull request May 31, 2012
[TIMOB-7773]iOS: Back/close animation broken when a NavigationGroup is in a TabGroup
@sptramer sptramer merged commit c9a0c22 into tidev:master May 31, 2012
@vishalduggal vishalduggal deleted the timob-7773 branch October 27, 2014 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants