Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#14716 closed enhancement (fixed)

dojox.mobile.parser should support data-dojo-mixins

Reported by: ykami Owned by: ykami
Priority: high Milestone: 1.8
Component: DojoX Mobile Version: 1.7.1
Keywords: Cc:
Blocked By: Blocking:

Description

data-dojo-mixins was introduced in dojo.parser (#14342). Mobile applications also need this functionality.

Change History (3)

comment:1 Changed 8 years ago by ykami

Resolution: → fixed
Status: new → closed

In [27729]:

Fixes #14716 !strict Added support for data-dojo-mixins, which is the same function as the one in dojo.parser (#14342). Renamed args to options.

comment:2 Changed 8 years ago by ykami

Milestone: tbd → 1.8
Priority: undecided → high

comment:3 Changed 8 years ago by bill

In [28094]:

Store parser's lowercase --> mixed case map as property of each widget's constructor, rather than in a global _nameMap[] hash.

This is necessary for #14960, since the constructor may have no string ID, and also fixes a subtle bug with the data-dojo-mixins feature #14716: for a node with data-dojo-type="A" and data-dojo-mixins="B, C, D", the parser was still doing a lookup as _nameMap[A], thus not finding the mixed-case methods from B, C, D.

Refs #14716, #14960 !strict.

Note: See TracTickets for help on using tickets.