Opened 8 years ago

Closed 7 years ago

#14674 closed defect (wontfix)

dojox/mobile/app/_event.js does not replace dojo.connect correctly

Reported by: Christian Schröder Owned by:
Priority: undecided Milestone: tbd
Component: DojoX Mobile Version: 1.7.1
Keywords: Cc:
Blocked By: Blocking:

Description

dojox/mobile/app/_event.js does replace the dojo.connect method to implement the event type conversion from mouse events to touch events. I guess with the AMD release the replacement of dojo connect is failing now.

the given patch fixes this:

Index: dojox/mobile/app/_event.js
===================================================================
--- dojox/mobile/app/_event.js	(revision 6329)
+++ dojox/mobile/app/_event.js	(working copy)
@@ -105,8 +105,8 @@
 	};
 
 }
-dojo._oldConnect = dojo._connect;
-dojo._connect = function(obj, event, context, method, dontFix){
+dojo._oldConnect = dojo.connect;
+dojo.connect = function(obj, event, context, method, dontFix){
 	event = dojox.mobile.app.eventMap[event] || event;
 	if(event == "flick" || event == "onflick"){
 		if(dojo.global["Mojo"]){

Change History (2)

comment:1 Changed 8 years ago by bill

Component: GeneralDojoX Mobile
Status: newassigned

comment:2 Changed 7 years ago by bill

Resolution: wontfix
Status: assignedclosed

dojox/mobile/app is deprecated in favor of dojox/app, and presumably will be removed in 2.0, so seems like we should mark this as wontfix.

Note: See TracTickets for help on using tickets.