Fixes missing touch.release events on some dojox/mobile widgets by al…

…ways listening to touch.release on the toplevel document. Thanks Eric Durocher (IBM, CCLA) git-svn-id: http://svn.dojotoolkit.org/src/dojox/trunk@30664 560b804f-0ae3-0310-86f3-f6aa0a117693
dojo · Feb 22, 2013 · b3be799 · b3be799
1 parent 8ab24df
commit b3be799
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/mobile/Switch.js b/mobile/Switch.js
@@ -169,7 +169,7 @@ define([
 			if(!this._conn){
 				this._conn = [
 					this.connect(this.inner, touch.move, "onTouchMove"),
-					this.connect(this.inner, touch.release, "onTouchEnd")
+					this.connect(win.doc, touch.release, "onTouchEnd")
 				];
 			}
 			this.touchStartX = e.touches ? e.touches[0].pageX : e.clientX;

diff --git a/mobile/_EditableIconMixin.js b/mobile/_EditableIconMixin.js
@@ -4,6 +4,7 @@ define([
 	"dojo/_base/declare",
 	"dojo/_base/event",
 	"dojo/_base/lang",
+	"dojo/_base/window",
 	"dojo/dom-geometry",
 	"dojo/dom-style",
 	"dojo/touch",
@@ -12,7 +13,7 @@ define([
 	"./sniff",
 	"./viewRegistry",
 	"./_css3"
-], function(array, connect, declare, event, lang, domGeometry, domStyle, touch, registry, IconItem, has, viewRegistry, css3){
+], function(array, connect, declare, event, lang, win, domGeometry, domStyle, touch, registry, IconItem, has, viewRegistry, css3){
 
 	// module:
 	//		dojox/mobile/_EditableIconMixin
@@ -131,7 +132,7 @@ define([
 			if(!this._conn){
 				this._conn = [
 					this.connect(this.domNode, touch.move, "_onTouchMove"),
-					this.connect(this.domNode, touch.release, "_onTouchEnd")
+					this.connect(win.doc, touch.release, "_onTouchEnd")
 				];
 			}
 			this._touchStartPosX = e.touches ? e.touches[0].pageX : e.pageX;

diff --git a/mobile/_EditableListMixin.js b/mobile/_EditableListMixin.js
@@ -5,13 +5,14 @@ define([
 	"dojo/_base/connect",
 	"dojo/_base/declare",
 	"dojo/_base/event",
+	"dojo/_base/window",
 	"dojo/dom-class",
 	"dojo/dom-geometry",
 	"dojo/dom-style",
 	"dojo/touch",
 	"dijit/registry",
 	"./ListItem"
-], function(array, connect, declare, event, domClass, domGeometry, domStyle, touch, registry, ListItem){
+], function(array, connect, declare, event, win, domClass, domGeometry, domStyle, touch, registry, ListItem){
 
 	// module:
 	//		dojox/mobile/EditableRoundRectList
@@ -111,7 +112,7 @@ define([
 			if(!this._conn){
 				this._conn = [
 					this.connect(this.domNode, touch.move, "_onTouchMove"),
-					this.connect(this.domNode, touch.release, "_onTouchEnd")
+					this.connect(win.doc, touch.release, "_onTouchEnd")
 				];
 			}
 			this._pos = [];