item passed in event differs between Mouse and Touch for dojos/calendar/Calendar

[sgevers]

The item passed to the onItemClick event from the Touch mixin does not contain the same properties as the one passed from the Mouse mixin. Looking at the source, the Mouse mixin uses the function renderItemToItem function to create the item that is passed to the event handler (Mouse.js: line 63). The Touch mixin uses itemToRenderItem to create the item that is passed to the event handler (Touch.js: line 317). The latter function appears to be intended for display purposes and does not contain all of the properties found on the original item in the store.

I'm attaching an example. The CalendarBug?.js file is a copy of dojox/calendar/Calendar.js with Touch added to the set of base classes mixed in at line 32. Cal.js is a simple widget that creates an instance of CalendarBug? and adds a handler for itemClicked. The calendar is created with a Memory store that contains one item. The one item has a property named "extraInfo". The handler simply creates an alert that shows the value of the item's extraInfo property.

When this example is run on a desktop browser, and the item is clicked, the alert contains the string "extraInfo" (which is the value of the extraInfo property). When run on a mobile device, the alert contains "undefined"

[sgevers]

HTML for example

[sgevers]

Driver JS

[sgevers]

dojox/calendar/Calendar with Touch mixed in

[cjolif]

This is ​https://github.com/damiengarbarino/dojo-calendar/issues/45

This has been fixed.