#15572 closed defect (fixed)
Css theme: display warning on all valid* fields even if is valid
Reported by: | Mathevet julien | Owned by: | Douglas Hays |
---|---|---|---|
Priority: | undecided | Milestone: | 1.8 |
Component: | Dijit - Form | Version: | 1.8.0b1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
In dialog box, if there is one non valid fields, all other valid* fields display a warning img.
This is true for all non claro theme.
See patch of test file to see bug. Enter a non valid date in first dialog.
Attachments (3)
Change History (16)
Changed 9 years ago by
Attachment: | testDijitErrorinDialog.diff added |
---|
comment:1 Changed 9 years ago by
Changed 9 years ago by
Attachment: | patchNihilo.diff added |
---|
comment:2 Changed 9 years ago by
I submit nihilo exemple fix. But is different of Claro.
I remark also that validation input in claro seem reduce due to a large validationContainer... seem a different bug
comment:3 Changed 9 years ago by
I'll take a look but I'm confused by your test file. It only has one ValidationTextBox, so how could it demonstrate a problem with multiple warning images? Aren't you saying that multiple ValidationTextBoxes all suddenly become marked as invalid?
comment:4 Changed 9 years ago by
Milestone: | tbd → 1.8 |
---|---|
Status: | new → assigned |
Oh I see, typing an invalid date in the DateTextBox makes the ValidationTextBox also display an error.
The dijitError class is showing up on Dialog since Dialog extends _FormWidget (see [9768]), and also _CssStateMixin. It seems a bit strange for a Dialog itself to be marked as an error, but I guess it's not a problem per se. I'll fix the CSS like you suggested.
comment:5 Changed 9 years ago by
Component: | Dijit → Dijit - Form |
---|---|
Owner: | changed from bill to Douglas Hays |
Actually Doug, probably you should take this one. The dijitError class is used in all the themes except claro, and it appears in dijit.css itself. I guess it's just a question of using dijitTextBoxError and dijitSelectError instead.
Alternately you could argue that the dijitError class shouldn't be getting applied to any container classes like Dialog, but it seems fragile to depend on that.
comment:6 Changed 9 years ago by
Please review the attached patch. I proliferated the dijitValidationTextBox class name so that CSS could be inherited without having to duplicate rules (especially by custom widgets). This class was already being added to plain ValidationTextBox widgets but none of the subclasses which seemed wrong.
comment:7 Changed 9 years ago by
I'm just a bit confused. I applied your patch, and tried DateTextBox: it seems to display an error fine, but it doesn't have the dijitValidationTextBox class on it.
PS: ignore this comment, I was testing against the wrong source tree, without your patch applied.
comment:8 Changed 9 years ago by
Another thing also: With claro theme with my patch for test file, when you enter a non valid date the content of validationTextBox is reduce: reduce to 4 letters. Then if you set a valid date other letters appears.
Seem the dijitValidationIcon is too large or something ike that
comment:10 Changed 9 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Thanks for the fix. I'm seeing errors in tests/form/robot/Select.html now though (tested on IE8), starting with [29077].
This is due to dijitError set on dialog. In claro there are other class that override it.