Opened 9 years ago
Closed 8 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 9 years ago by
Component: | General → DojoX Mobile |
---|---|
Status: | new → assigned |
comment:2 Changed 8 years ago by
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
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.