[TIMOB-25370] Android: Scrolling up with Swipe Refresh Control doesn't work
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | Critical |
Status | Closed |
Resolution | Duplicate |
Resolution Date | 2017-11-06T17:03:04.000+0000 |
Affected Version/s | n/a |
Fix Version/s | n/a |
Components | Android |
Labels | n/a |
Reporter | Mohamad Alhajj |
Assignee | Joshua Quick |
Created | 2017-10-04T12:08:57.000+0000 |
Updated | 2017-11-06T17:03:04.000+0000 |
Description
After upgrading to Titanium SDK 6 we faced a cretical issue with all ListView/TableViews and ScrollView, that uses this module (https://github.com/raymondkam/Ti.SwipeRefreshLayout) for pull to refresh,
The issue is as follows:
* Scrol down in a scrollview for example.
* Remove your finger.
* Try to scroll up, you cannot, since the module will capture the event first and refreshes the list.
Before we were using Titanium SDK 6.1.2.GA and it was working as expected.
Thanks,
[~malhajj], we've added official RefreshControl support to ListViews and TableViews on Android in Titanium v6.2.0. Perhaps its conflicting with the 3rd party module you're using? I recommend that you switch over to our RefreshControl in 6.2.0. Just note that Android ScrollViews do not currently support our RefreshControl. You can assign a RefreshControl to it, but it'll no-op and log a warning that it's not currently supported. The 3rd party module you're using isn't made by us. So, I recommend that you contact the author about this issue. Titanium's implementation of the vertical ScrollView has been changed to use Google's "NestedScrollView" Java class and I suspect this module needs to be changed to compensate for it.
Hello Joshua, Thanks for the clarification, but is there a ticket related to supporting Refresh Control in android scrollview so that we can track Sent from JIRA Mobile
[~malhajj], yes, we've written up a ticket here: [TIMOB-25197]
[~malhajj], Android RefreshControl support will be added to ScrollViews in Titanium 6.3.0. We'll be releasing a release candidate of 6.3.0 soon. This should solve your problem and you'll no longer need to use that SwipeRefreshLayout module anymore.
That’s what I’m talking about Thanks a lot Joshua
The 6.3.0 RC has been released... http://www.appcelerator.com/blog/2017/10/rc-release-of-titanium-sdk-6-3-0-cli-6-30/ If you can confirm that this switching to our RefreshControl feature on Android solves your scrolling issues, then that would be great. Thanks.
I’ve got the time and checked this, I’m glad to say its working for all my cases in listview, Scrollview and besides its also working great while using the module mentioned. Titanium SDK 6.3.0.GA was used Well done guys and thanks for these great efforts Sent from JIRA Mobile
Thanks for the feedback [~malhajj]!
[TIMOB-25197] resolves this issue.