{ "id": "62611", "key": "TIMOB-1979", "fields": { "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "subtask": false }, "project": { "id": "10153", "key": "TIMOB", "name": "Titanium SDK/CLI", "projectCategory": { "id": "10100", "description": "Titanium and related SDKs used in application development", "name": "Client" } }, "fixVersions": [ { "id": "11230", "name": "Release 1.5.0 M05", "archived": true, "released": true, "releaseDate": "2010-12-13" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2011-04-17T01:57:43.000+0000", "created": "2011-04-15T03:07:05.000+0000", "priority": { "name": "Medium", "id": "3" }, "labels": [ "android", "defect", "regression" ], "versions": [], "issuelinks": [], "assignee": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "updated": "2011-04-17T01:57:43.000+0000", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "10202", "name": "Android", "description": "Android Platform" } ], "description": "{html}

1.Install GG, launch the app
\n2.tap the video game icon
\n3.in the search, type in \"killzone\"
\n4.tap killzone 2 and let the page load.
\n5.Use your finger to tap the \"Guru\" row
\n6.Tap the table rows below as well

\n

results: using tap the guru row fails to show highlight (works\nfine using trackball), the rows below have intermittent highlight\nbehavior.

\n

As Per Bill \"It was one of the commits in between your Tuesday\nbuild (1.4.2 80434d) and my fix (1.4.2.5935b3) that caused the\nproblem with the row highlight\" (Thomas added build numbers to\nBills quote)

\n

\nhttp://github.com/appcelerator/titanium_mobile/commit/d074f62eaf9

{html}", "attachment": [], "flagged": false, "summary": "Android - row highlight fails with touch", "creator": { "name": "thomashuelbert", "key": "thomashuelbert", "displayName": "Thomas Huelbert", "active": true, "timeZone": "America/Los_Angeles" }, "subtasks": [], "reporter": { "name": "thomashuelbert", "key": "thomashuelbert", "displayName": "Thomas Huelbert", "active": true, "timeZone": "America/Los_Angeles" }, "environment": null, "comment": { "comments": [ { "id": "127141", "author": { "name": "billdawson", "key": "billdawson", "displayName": "Bill Dawson", "active": true, "timeZone": "Europe/Berlin" }, "body": "{html}

Don: note that it's not that row highlighting is totally broken.\nIf I create a table that has rows with just the title\nproperty set, then it works. But the affected application has an\nimage view and multiple labels that are .add'd to the\nrow. The highlighting works in the commit just before your commit\n(http://github.com/appcelerator/titanium_mobile/commit/d074f62eaf9),\nbut then not after.

\n

I'll try to make a more general failcase so you don't have to go\nwading through their code.

{html}", "updateAuthor": { "name": "billdawson", "key": "billdawson", "displayName": "Bill Dawson", "active": true, "timeZone": "Europe/Berlin" }, "created": "2011-04-15T03:07:05.000+0000", "updated": "2011-04-15T03:07:05.000+0000" }, { "id": "127142", "author": { "name": "billdawson", "key": "billdawson", "displayName": "Bill Dawson", "active": true, "timeZone": "Europe/Berlin" }, "body": "{html}

Here's the failcase I promised. All in one app.js:

\n
\n\n/*global Ti, Titanium, alert, JSON */\nTitanium.UI.setBackgroundColor('#000');\nvar win = Titanium.UI.createWindow({  \n    title:'Test',\n    backgroundColor:'#fff',\n    fullscreen: true,\n    exitOnClose: true\n});\n\n// Rows with just \"title\" -- highlighting works fine.\nvar plainrows = [];\nfor (var i = 0; i < 3; i++) {\n    plainrows.push(\n        Ti.UI.createTableViewRow( {title: 'row ' + i} )\n    );\n}\n\nwin.add( Ti.UI.createTableView( {\n    data: plainrows,\n    top: 0, left: 0, right: 0, height: 150\n} ) );\n\n\n// Rows with views in them -- highlighting not working\n// since commit d074f62eaf9.\n\nvar viewrows = [];\nfor (i = 0; i < 3; i++) {\n    var l = Ti.UI.createLabel({\n        text: 'row ' + i,\n        top: 0, left: 0, width: 100, height: 40\n    });\n    var tr = Ti.UI.createTableViewRow();\n    tr.add(l);\n    viewrows.push(tr);\n}\n\nwin.add( Ti.UI.createTableView( {\n    data: viewrows,\n    top: 160, left: 0, right: 0, height: 150\n} ) );\n\nwin.open();\n
\n

If you click on the labels in the second set of rows, no\nhighlighting will occur, whereas it did before commit \nd074f62eaf9 (try building with 1.4.0 to see it work). I'm\nafraid this is probably high priority because it's a very\nnoticeable regression in GG, for example.

{html}", "updateAuthor": { "name": "billdawson", "key": "billdawson", "displayName": "Bill Dawson", "active": true, "timeZone": "Europe/Berlin" }, "created": "2011-04-15T03:07:05.000+0000", "updated": "2011-04-15T03:07:05.000+0000" }, { "id": "127143", "author": { "name": "billdawson", "key": "billdawson", "displayName": "Bill Dawson", "active": true, "timeZone": "Europe/Berlin" }, "body": "{html}

I should have tagged this a regression.

{html}", "updateAuthor": { "name": "billdawson", "key": "billdawson", "displayName": "Bill Dawson", "active": true, "timeZone": "Europe/Berlin" }, "created": "2011-04-15T03:07:06.000+0000", "updated": "2011-04-15T03:07:06.000+0000" }, { "id": "127144", "author": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "body": "{html}

Duplicate of #1509

{html}", "updateAuthor": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2011-04-15T03:07:06.000+0000", "updated": "2011-04-15T03:07:06.000+0000" }, { "id": "127145", "author": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "body": "{html}

I incorrectly marked this as a dup of #1509.\n#2101 is however a duplicate of this one.

{html}", "updateAuthor": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2011-04-15T03:07:06.000+0000", "updated": "2011-04-15T03:07:06.000+0000" }, { "id": "127146", "author": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "body": "{html}

For consideration in M05

{html}", "updateAuthor": { "name": "dthorp", "key": "dthorp", "displayName": "Don Thorp", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2011-04-15T03:07:06.000+0000", "updated": "2011-04-15T03:07:06.000+0000" }, { "id": "127147", "author": { "name": "mculpepper", "key": "mculpepper", "displayName": "Marshall Culpepper", "active": true, "timeZone": "America/Los_Angeles" }, "body": "{html}

Labels are now clickable by default, which causes the event\nhandler in Label to not send the event up to the TableViewRow. You\ncan disable this behavior by setting \"touchEnabled: false\" in the\ncreation proxy for each label that you want to click through to the\nrow. Marking as resolved.

{html}", "updateAuthor": { "name": "mculpepper", "key": "mculpepper", "displayName": "Marshall Culpepper", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2011-04-15T03:07:07.000+0000", "updated": "2011-04-15T03:07:07.000+0000" } ], "maxResults": 7, "total": 7, "startAt": 0 } } }