{ "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
results: using tap the guru row fails to show highlight (works\nfine using trackball), the rows below have intermittent highlight\nbehavior.
\nAs 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
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.
I'll try to make a more general failcase so you don't have to go\nwading through their code.
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
\nIf 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.
I should have tagged this a regression.
Duplicate of #1509
For consideration in M05
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.