Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-25190] Android: touchEnabled is not true by default if we have a property specified for the tableViewRow

GitHub Issuen/a
TypeBug
PriorityNone
StatusClosed
ResolutionFixed
Resolution Date2017-08-25T18:23:42.000+0000
Affected Version/sRelease 7.0.0, Release 6.2.0
Fix Version/sn/a
ComponentsAndroid
Labelsqe-6.2.0, regression
ReporterLokesh Choudhary
AssigneeEric Merriman
Created2017-08-24T02:12:13.000+0000
Updated2017-08-25T18:23:42.000+0000

Description

*This is a regression. Worked fine with 6.1.2.GA* Docs : http://docs.appcelerator.com/platform/latest/#!/api/Titanium.UI.TableViewRow-property-touchEnabled

Steps to reproduce:

1. Use the below code in your app.js.
var win = Ti.UI.createWindow({
   });
  var data=[];
   var table = Ti.UI.createTableView({
        top:0,
        width: Ti.UI.FILL,
        height: Ti.UI.FILL
   }); 
   var row = Ti.UI.createTableViewRow({
	    width: Ti.UI.FILL,
	    // touchEnabled: true,   (setting touchEnabled: true works fine) 
	    height: 64                     (if height or any other property is used the we loose touch)
	});
data.push(row);

table.addEventListener("click", function (e) {
    Ti.API.warn("Clicked: " + JSON.stringify(e.rowData));
});

table.data = data; 
win.add(table);
win.open();
2. Build for android device/emulator. 3. After the app launches tap on the tableviewrow.

Actual results:

1. No click event is fired. 2. If no properties are used for the table view row then click works fine, but if we have a single property the touch/click stops working & needs touchEnabled to be set to true to get the touch/click.

Expected result:

1. Click event is fired.

Comments

  1. Joshua Quick 2017-08-24

    [~lchoudhary], [~ybanev], It sounds like fix for [TIMOB-11856] is what caused this regression.
  2. Lokesh Choudhary 2017-08-25

    This issue is fixed with these PR's: master: https://github.com/appcelerator/titanium_mobile/pull/9353 6_2_X: https://github.com/appcelerator/titanium_mobile/pull/9354
  3. Lokesh Choudhary 2017-08-25

    FR passed for both master & backport.
  4. Lokesh Choudhary 2017-08-25

    PR's merged.
  5. Lokesh Choudhary 2017-08-25

    Closing this as the related ticket has the fix for this & its fixed.

JSON Source