dojo.attr(node, "readonly", ...) not working in firefox / safari

[s_i_z_o]

In test page for nightly builds:

​http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/Form.html

Read-only widget on the bottom of the page is not really readonly in firefox 2.0.0.20. Value can be changed and is posted on submit. It is working in IE 7.x, didn't check other browsers.

[bill]

Good catch. It happens in FF3 and safari too. The problem is in dojo.attr(), you can try this in firebug from that page to see:

dojo.attr("dijit_form_DateTextBox_0", "readonly", true)

The problem started in [12332] for #5781. The root issue is that when setting the readonly property via

node.readOnly = true

"readOnly" needs to be in camel case, not lower case. Previously the dojo.attr() code called node.setAttribute(), which is case insensitive, but after [12332] it started assigning the property directly.

I guess fixAttr() should be changed to always convert readonly to readOnly, for all browsers?

[bill]

Reference page: ​http://www.w3schools.com/htmldom/prop_text_readonly.asp

[bill]

Also, Cougar pointed out that this is basically a dup of #7896, except for FF/safari rather than IE8. Anyway, needs to be tested on all browsers.

[Douglas Hays]

(In [16225]) Fixes #8344 !strict. Change readonly to readOnly for all browsers.