Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#14716 closed enhancement (fixed) 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:


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

Change History (3)

comment:1 Changed 10 years ago by ykami

Resolution: fixed
Status: newclosed

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 10 years ago by ykami

Milestone: tbd1.8
Priority: undecidedhigh

comment:3 Changed 10 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.