[TIMOB-2856] Android: Touchend Events are broken
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | Low |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2011-04-18T13:49:27.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 1.7.0, Sprint 2011-10, Release 1.6.2 |
Components | Android |
Labels | android, bug, defect, klist, patch, release-1.6.2, release-1.7.0, reported-1.6.0, rplist, touchend |
Reporter | Alan Leard |
Assignee | Bill Dawson |
Created | 2011-04-15T03:31:17.000+0000 |
Updated | 2011-04-18T13:49:27.000+0000 |
Description
Touchend events seem to be broken on buttons and views on Android.
Bug possibly located in TiUIView.java. User has debugged and created a patch.
Ticket Reference: http://developer.appcelerator.com/helpdesk/view/66211">http://developer.appcelerator.com/helpdesk/view/66211
GitHub Patch created by user:https://github.com/appseleration/titanium_mobile/commit/ad24c453be4bd3c354621dbe3ea9e7db28fa0cdb">https://github.com/appseleration/titanium_mobile/commit/ad24c453be4...
(from [3d339534f6d4f8e414ec1d6c9da571aa798b6fc4]) Use correct coordinates when checking touch up versus touch cancel in TiUIView. Kudos and thanks to Nils Domrose for pointing it out.[#2856 state:fixed-in-qa] https://github.com/appcelerator/titanium_mobile/commit/3d339534f6d4f8e414ec1d6c9da571aa798b6fc4"> https://github.com/appcelerator/titanium_mobile/commit/3d339534f6d4...
For QA tester: you can see the fail case and fix in Kitchen Sink. Go to the Base UI - Window Events test. If you tap anywhere on that label down below, the one that reads "try to trigger each event", you'll see that your click produces a touchstart and a touchcancel event. <-- that's the fail case, because it doesn't make sense. A "touchcancel" is when you touch-and-hold on a view (such as that label), move off of the view, then lift your finger. If you just tap a view, you should be firing touchstart->touchend, not touchstart->touchcancel.
Tested with Titanium SDK version: 1.7.0 (03/15/11 11:42 1f6dbfe) on
Nexus S 2.3.2
Emulator 2.1
Added to stable branch 1_6_X
Setting affectedVersion. Not possible w/o reopening.
updated fixed versions, closing.