Current custom data-* prefix parser has some degradation

[youngho]

Current custom data-* prefix parser has some degradation. Please refer to the attached test file.

when run test file, I got

missing : after property id ​http://localhost/js/dojo-src/dojo/tests/parser_fail.html Line 19

[Fredde]

Looks like you are mixing single and double quotes on line 20 in the test file.

data-dojo-props:'onClick:function(){ console.log("clicked simple"); }, iconClass: 'plusIcon', value: 'Create' "

should be:

data-dojo-props:"onClick:function(){ console.log('clicked simple'); }, iconClass: 'plusIcon', value: 'Create' "

[bill]

Right, please reopen if there's still an issue after fixing the quotes.

data-dojo-props and data-dojo-types also need quotes or otherwise it's a subtraction of three variables.

[youngho]

Sorry I should be reopen again,

After fixing the typo data-dojo-props: "... ", Still it doesn't work at all.

[bill]

OK, please attach the updated testcase.

[youngho]

typo fixed test file

[youngho]

Hello Bill,

I updated the test file.

[bill]

Your test file is still invalid, as I said above data-dojo-props and data-dojo-types also need quotes or otherwise it's a subtraction of three variables.

[youngho]

2nd fixed test file

[youngho]

Hello Bill,

Sorry for missing your comment. I updated again my test file.

[bill]

OK that's better, it's syntactically valid javascript. But from the code it looks like instantiate() expects a hash for it's mixin parameter like:

this.instantiate(myNodes, {
   "data-dojo-type": "dijit.form.Button",
   intParam: 5,
   stringParam: "hi"
})

As to what API makes sense, that's up for debate. Is the above API inconvenient for you? It does seem weird compared to how parser works declaratively.

[youngho]

Also in IE8, the custom className didn't set at all. With FF, I can see the custom class name on the widget, but IE8 it didn't.

I attached another test file for that.

[youngho]

IE8, the custom class name didn't set at all

[bill]

That test is invalid because it's using "className" instead of "class" (note that you need quotes around "class" since it's a reserved word"), but if you still have problems after fixing that please open a new ticket about it.

[youngho]

I confirme test parser_fail_IE8.html is wrong. After bill's suggestion applied, IE8 works correctly.