bug fix to play nice with sencha touch 2.3

[fredj]

We have included ol3 map in a sencha touch 2.3 mobile application. Within the mobile app, the map crash as soon as we click on it. We have been able to pin point the problem (and do a dummy fix) with the ol-whitespace.js file.

OL3 version: 3.0.0-beta.5
File: ol-whitespace.js

Line: 25856
var touch = event.changedTouches.item(0);
replaced by
var touch = event.changedTouches[0];

No more crash, the click event is now propagated successfully to the sencha app handlers.

[fredj]

Are you experimenting the same issue without sencha? eg. http://ol3js.org/en/master/examples/mobile-full-screen.html

This is weird; the item function is standard: https://developer.mozilla.org/en-US/docs/Web/API/TouchList.item

[fredj]

Sencha touch is probably building or changing the changedTouches property using a plain Array

[elemoine]

Sencha touch is probably building or changing the changedTouches property using a plain Array

Yes that's it, I've seen this in the past as well.

[fredj]

PR attached, please review

[tsauerwein]

changedTouches is also used in the pointer events module, but there we use the array accessors [], so it is fine.

[fredj]

@tsauerwein ok, thanks for the info

[elemoine]

+1 as long as using [] is the same as using the item function.

[fredj]

Yes it's the same: "You may also use standard array syntax to access items in the list."

[jadetr]

Are you experimenting the same issue without sencha?
@fredj As expected by reading the comments here, without sencha everything works fine.

Fix quickly promoted in the main branch, nice job!
Next time I'll try to fork, dig in the source and push a fix :)