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 9827 Android: Translate animations of invisible views may be delayed #2556
Conversation
…ff-screen) should invalidate parent to avoid Android delaying the animation start time.
@@ -290,26 +293,48 @@ public AnimationSet render(TiViewProxy viewProxy, View view, int x, int y, int w | |||
TiDimension optionTop = null, optionBottom = null; | |||
TiDimension optionLeft = null, optionRight = null; | |||
TiDimension optionCenterX = null, optionCenterY = null; | |||
|
|||
|
|||
TiUIView tiView = viewProxy.peekView(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably do a null check for tiView
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nm, ignore this comment, I thought you were calling tiView.getBottom below =)
Code reviewed, left one comment. |
Good point, lemme fix that |
Ready for re-review. |
Code reviewed and functionally tested. Request Accepted |
Timob 9827 Android: Translate animations of invisible views may be delayed
See JIRA Testing Notes.
NOTE Please also resolve TIMOB-9954 with this. The test app in TIMOB-9827 includes a part called "extended tests", which will show that 9827 is fixed. You can review 9827 description to see what I mean.