ValidateTextbox inhibits custom css to control appearance

[[email protected]…]

The 'highlight' method of ValidateTextbox? sets the value of the css classname for the input element to one of dojoValidateEmpty, dojoValidateValid or dojoValidateInvalid. This overrides the behaviour of being able to set a class or className attribute in the markup to control appearance. This affects a large number of derivitive controls including all the spinner controls.

Furthermore, the dojoValidate* css classes are not defined anywhere.

It appears that an earlier behaviour was to simply adjust the background color of the input element. reverting to this would be a possibility.

[tk]

Terry,

Would you agree that using add/removeClass would solve this problem? I'm not sure to what extent you are modifying with a preset Class, that is why I ask.

[[email protected]…]

Thanks Karl, add/removeClass would be a good solution.

[Karl Tiedt]

(In [5986]) References #1590 (not going to close until verified fixed with reporter)

adds updateClass() in validationTextbox that removes all validation classes and adds in the new one using add/removeClass versus setClass.

[tk]

Terry,

I updated with an add/removeClass function, its late here and I dont have an old copy of Dojo setup at this time, can you verify this fixes the problem?

If not I'll be checking it tommorow :)

-Karl

[[email protected]…]

Karl

The mods you made don't compile (I tested with IE6). The problem is in the new updateClass method. It has an argument named 'class'. ECMA-262 says this is a reserved word.

I tried changing 'class' to 'cls' and all is well. Achieves the desired result wrt how we are using it.

Terry.

[Karl Tiedt]

(In [5994]) Fixed reserved word class in ValidationTextbox?.js fixes #1590

[(none)]

Milestone 0.4 deleted