Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-25411] Android: ScrollView does not fire events

GitHub Issuen/a
TypeBug
PriorityHigh
StatusClosed
ResolutionFixed
Resolution Date2017-10-23T22:30:55.000+0000
Affected Version/sRelease 6.3.0
Fix Version/sRelease 6.3.0
ComponentsAndroid
Labelsqe-6.3.0, regression
ReporterLokesh Choudhary
AssigneeJoshua Quick
Created2017-10-19T01:27:24.000+0000
Updated2017-10-26T22:10:48.000+0000

Description

*This is a regression. Not seen with 6.2.2.GA.*

Steps to reproduce:

1. Use the code below in your app.js:
var win = Ti.UI.createWindow();

var scrollView = Ti.UI.createScrollView({
        width: 300,
        height: 300,
        contentWidth: "auto",
        contentHeight: "auto",
        backgroundColor: 'red'
    });

scrollView.addEventListener('click', function(e) {
        alert('ScrollView Clicked');
});

scrollView.addEventListener('dblclick', function(e) {
        alert('Double Clicked');
});

scrollView.addEventListener('doubletap', function(e) {
        alert('Double Tapped');
});

scrollView.addEventListener('focus', function(e) {
        alert('Focus Fired');
});

scrollView.addEventListener('longclick', function(e) {
        alert('longclick Fired');
});

scrollView.addEventListener('longpress', function(e) {
        alert('longpress Fired');
});

scrollView.addEventListener('pinch', function(e) {
        alert('pinch Fired');
});

scrollView.addEventListener('postlayout', function(e) {
        alert('postlayout Fired');
});

scrollView.addEventListener('scroll', function(e) {
        alert('scroll Fired');
});

scrollView.addEventListener('swipe', function(e) {
        alert('swipe Fired');
});

win.add(scrollView);
win.open();
2. Build the app for android device/emulator.

Actual results:

1. Only postlayout event is fired & no other event is fired.

Expected results:

1. All the specified events should be fired.

Comments

  1. Joshua Quick 2017-10-23

    PR (6.3.x): https://github.com/appcelerator/titanium_mobile/pull/9543 PR (master): https://github.com/appcelerator/titanium_mobile/pull/9545
  2. Lokesh Choudhary 2017-10-23

    FR Passed for both PR's. Master merged, waiting for merge to get enabled for 6.3.0.
  3. Lokesh Choudhary 2017-10-26

    PR for backport merged.
  4. Lokesh Choudhary 2017-10-26

    Verified the fix in SDK 6.3.0.v20171026120401 & 7.0.0.v20171025141436. Closing. Studio Ver: 4.10.0.201709271713 OS Ver: 10.12.3 Xcode Ver: Xcode 8.3.3 Appc NPM: 4.2.10-2 Appc CLI: 6.3.0-master.15 Ti CLI Ver: 5.0.14 Alloy Ver: 1.10.6 Node Ver: 7.10.1 Java Ver: 1.8.0_101 Devices: ⇨ google Nexus 5 — Android 6.0.1 ⇨ google Pixel — Android 7.1.1

JSON Source