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-17572] iOS: Fix activity indicator animation in list items #9717
Conversation
@@ -91,6 +94,21 @@ - (void)prepareForReuse | |||
{ | |||
RELEASE_TO_NIL(_dataItem); | |||
[super prepareForReuse]; | |||
|
|||
#ifdef USE_TI_UIACTIVITYINDICATOR |
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.
Is this issue not in TiUITableView ?
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.
The issue was only described in Ti.UI.ListView because the developer can re-animate the indicator inside the Ti.UI.TableView as it's children are not hold virtually. As I didn't see any comparable issue on JIRA, I only made it for this one for now (and because we recommend to use Ti.UI.ListView for performant usages).
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.
Okey.
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.
Cr and FT passed.
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.
Passed FR. Details in master PR.
I added "in-qe-testing" label back as per our discussion in standup. |
Generated by 🚫 dangerJS |
@hansemannn "Merge" button is still disabled. Should this one be force merged? |
@mukherjee2 Random Android unit-test failure, no idea who should merge that. I can‘t, sorry. |
JIRA: https://jira.appcelerator.org/browse/TIMOB-17572