[TIMOB-3012] iOS Kitchen Sink: Table View Pull to Refresh Broken by #2975
| GitHub Issue | n/a |
|---|---|
| Type | Bug |
| Priority | Trivial |
| Status | Closed |
| Resolution | Duplicate |
| Resolution Date | 2011-04-15T03:34:45.000+0000 |
| Affected Version/s | n/a |
| Fix Version/s | Backlog |
| Components | iOS |
| Labels | defect, ios, kitchensink, regression, reported-1.6.0, rplist |
| Reporter | Dawson Toth |
| Assignee | Reggie Seagraves |
| Created | 2011-04-15T03:34:44.000+0000 |
| Updated | 2017-03-03T06:21:58.000+0000 |
Description
Problem
Our Pull to Refresh example no longer works because of the changes in #2975, due to scrollEnd firing after the table has stopped moving, as opposed to when the user lifts their finger.
Affected File
Resources/examples/table_view_pull_to_refresh.js, lines 144-157
Fix
The condition "&& e.contentOffset.y <= -65" on line 146 is not necessary, and is stopping the sample from working. Removing it will allow pull to fresh to function again.
Aesthetic Issue
Because scrollEnd is firing after the table stops moving, with the fix above, the table view will return to the top of the table after getting pulled down far enough and released. scrollEnd then fires, and the "Reloading..." status is shown, pushing the table back down. There may be another approach that would avoid the unnecessary table movement this results in.
Associated Helpdesk Ticket
http://developer.appcelerator.com/helpdesk/view/69331">http://developer.appcelerator.com/helpdesk/view/69331
Forgot to assign... let me know if this should get kicked to someone else, Kevin. I'm assigning to you based on a hunch.
Is there any events that still fire when a users finger is lifted?
duping against #3006
Closing issue due to time passed and irrelevance of the ticket.