[TIMOB-14931] MobileWeb: <allow-touch> option not properly toggling touch events for Ti.UI and ScrollableView

GitHub Issue	n/a
Type	Bug
Priority	Critical
Status	Closed
Resolution	Fixed
Resolution Date	2013-08-27T22:30:47.000+0000
Affected Version/s	Release 3.1.2
Fix Version/s	2013 Sprint 18 Core, 2013 Sprint 18, Release 3.1.3, Release 3.2.0
Components	MobileWeb
Labels	qe-closed-3.2.0, qe-testadded
Reporter	Chris Barber
Assignee	Chris Barber
Created	2013-08-21T18:37:07.000+0000
Updated	2014-02-24T23:20:47.000+0000

Description

The flag needs to override the touch detection, but we forgot to add this to titanium/Ti/UI/ScrollableView.js and titanium/Ti/UI.js. We simply need to change both instances of

useTouch = 'ontouchstart' in global,

useTouch = has('touch'),

Comments

Chris Barber 2013-08-27

Master pull request: https://github.com/appcelerator/titanium_mobile/pull/4610 3_1_X pull request: https://github.com/appcelerator/titanium_mobile/pull/4611
Chris Barber 2013-08-27

To test, build KitchenSink for mobile web, then open in a mobile browser such as an iOS device or iOS Simulator. Notice ScrollableView, ScrollView, TableView, Picker, and PickerColumn controls work as expected. Edit the tiapp.xml and set false in the tag. Rebuild KitchenSink and test again. Notice those 5 controls no longer respond to touches.
Olga Romero 2013-11-12

Tested and verified Fixed with: Appcelerator Studio, build: 3.2.0.201311120707 Titanium SDK, build: 3.2.0.v20131111174605 CLI 3.2.0 Alloy 1.3.0 KS 3.1.3 from studio samples

JSON Source