#3687 closed task (fixed)
use comma rather than semicolon as array separator
Reported by: | bill | Owned by: | Adam Peller |
---|---|---|---|
Priority: | high | Milestone: | 0.9 |
Component: | Dijit | Version: | 0.9 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
parser.js [9519] changed the array delimiter in the parser from semi-colon to comma, so that you declare array parameters like:
foo="1,2,3"
Should make same change for widget templates, so that dojoAttachEvent, etc. also uses commas:
dojoAttachEvent="onclick: _onClick, onkeyup: _onKeyUp"
Have to update the tests as well as the migration guide too.
Alternately/additionally arrays in the parser be specified as JS objects, like this?
foo="[1,2,3]"
We don't use arrays much in the code so I wonder if it makes sense to have a special case in the parser.
Change History (4)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
Owner: | changed from bill to Adam Peller |
---|
comment:3 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:4 Changed 14 years ago by
Resolution: | → fixed |
---|
Note: See
TracTickets for help on using
tickets.
Talked about it in today's meeting. No vociferous objections.