#1590 closed defect (fixed)
ValidateTextbox inhibits custom css to control appearance
Reported by: | Owned by: | tk | |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | Widgets | Version: | 0.3 |
Keywords: | Cc: | [email protected]…, [email protected]… | |
Blocked By: | Blocking: |
Description
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.
Change History (8)
comment:1 Changed 14 years ago by
Owner: | changed from bill to tk |
---|
comment:2 Changed 14 years ago by
Cc: | [email protected]… added |
---|---|
Status: | new → assigned |
comment:4 Changed 14 years ago by
comment:5 Changed 14 years ago by
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
comment:6 Changed 14 years ago by
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.
comment:7 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
(In [5994]) Fixed reserved word class in ValidationTextbox?.js fixes #1590
Terry,