PDA

View Full Version : [FIXED] CheckBox Readonly Doesn't Work



nathan.modrzewski
5 May 2008, 1:37 PM
When the readonly value is set to "true" in a checkbox or on a checkboxgroup, you can still check or uncheck the box with the mouse or space bar.

darrellmeyer
12 May 2008, 12:58 PM
Read only does not disable the check or radio, it just stops its value from being changed. You need to call disable() to stop user interaction. disable and enable was not being passed to the children when called on the group, this had been fixed. Change is in SVN.

gslender
12 May 2008, 2:13 PM
Darrell,

I think, for a checkbox, I'd expect the same behaviour as a textfield - which is that the user can "try" to modify the value but the UI does not change it... disabled is a completely different UI state and should indicate that the label & field is completely unavailable - ie grayed etc

Currently what you are proposing is confusing and worth either updating docs to reflect this inconsisentcy with textfield, or align textfield to do the same (ie allow the visual text field to be changed but the underlying value to remain static).

I'd prefer you change the checkbox to behave in a read-only manner -which means the check cannot be changed even though the control is not disabled (ie not grayed).

darrellmeyer
12 May 2008, 3:11 PM
All the read only method did was add the "READONLY" attribute to the HTML input element which does not cause the radio and check from being toggled. I have fixed this to stop the change by stopping the click event if the field is read only. Setting the field to read only will now stop the user from changing the value, although the user might be confused as the fields look enabled :). Change is in SVN.

gslender
12 May 2008, 6:09 PM
read only will now stop the user from changing the value, although the user might be confused as the fields look enabled :)

If I understand you correctly, this will be ok - read only does as it should - ie field cannot be changed but looks enabled - disabled will be visually disabled plus read-only (state not changable). I think this would be a correct UI representation.

The developer now has choice over if he wants the field read-only or fully disabled.