[TIMOB-26051] Android: Ti.UI.createRefreshControl.beginRefreshing() does not fire 'refreshstart'
GitHub Issue | n/a |
---|---|
Type | Improvement |
Priority | Critical |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2018-05-18T01:36:29.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 7.3.0 |
Components | Android |
Labels | android, parity, refreshcontrol |
Reporter | Gary Mathews |
Assignee | Gary Mathews |
Created | 2018-05-17T23:50:11.000+0000 |
Updated | 2018-06-19T16:46:06.000+0000 |
Description
-
beginRefreshing()
does not fire the refreshstart
event
*TEST CASE*
var window, control, listView;
window = Ti.UI.createWindow();
window.addEventListener('open', function () {
control.beginRefreshing();
});
control = Ti.UI.createRefreshControl();
listView = Ti.UI.createListView({
refreshControl: control
});
control.addEventListener('refreshstart', function () {
Ti.API.info('refreshstart');
setTimeout(function () {
Ti.API.info('ending...');
control.endRefreshing();
}, 2000);
});
control.addEventListener('refreshend', function () {
Ti.API.info('refreshend');
alert('DONE!');
});
window.add(listView);
window.open();
master: https://github.com/appcelerator/titanium_mobile/pull/10061
I've verified that this is the iOS behavior in Titanium 7.0.0. We need this change for parity. (Thanks Gary.)
Verified the fix with SDK 7.3.0.v20180618182516. Closing. Studio Ver: 5.1.0.201805300954 SDK Ver: 7.3.0.v20180618182516 OS Ver: 10.13.4 Xcode Ver: Xcode 9.4.1 Appc NPM: 4.2.13 Appc CLI: 7.0.4 Daemon Ver: 1.1.3 Ti CLI Ver: 5.1.1 Alloy Ver: 1.12.0 Node Ver: 8.9.1 NPM Ver: 5.5.1 Java Ver: 1.8.0_161 Devices: ⇨ google Nexus 5 --- Android 6.0.1 ⇨ google Nexus 6P --- Android 8.1.0 Emulator: ⇨ Android 4.1.2