Opened 12 years ago
Closed 12 years ago
#6005 closed enhancement (fixed)
[cla][patch] Form.validate() will indicate errors in the form
| Reported by: | nathan | Owned by: | Douglas Hays |
|---|---|---|---|
| Priority: | high | Milestone: | 1.1 |
| Component: | Dijit | Version: | 1.1b1 |
| Keywords: | Cc: | bill | |
| Blocked By: | Blocking: |
Description (last modified by )
Make Form.validate() turn all the invalid or required-but-blank fields yellow, and focus on the first invalid (or required-but-blank) field. It is a way to visually expose to the user that things aren't quite right.
Implementation notes: widget.isValid() won't be removed. widget.validate is extended to return the valid state (a boolean). Form.validate is a new method that calls widget.validate for each field - and in the event that one of them is invalid, it focuses on the first invalid.
Attachments (3)
Change History (9)
comment:1 Changed 12 years ago by
| Cc: | bill added |
|---|---|
| Owner: | set to Douglas Hays |
comment:2 Changed 12 years ago by
| Description: | modified (diff) |
|---|---|
| Owner: | changed from Douglas Hays to nathan |
| Summary: | [cla][patch] Suggestion: enhance dijit.form.Form.isValid → [cla][patch] Form.validate() will indicate errors in the form |
comment:3 Changed 12 years ago by
| Cc: | nathan added; [email protected]… removed |
|---|---|
| Status: | new → assigned |
comment:4 Changed 12 years ago by
| Cc: | nathan removed |
|---|---|
| Reporter: | changed from guest to nathan |
Changed 12 years ago by
| Attachment: | FormValidation.patch added |
|---|
Patch which adds: 1 - dijit.form.Form.validate(), 2 - returns the boolean state from ValidationTextBox?.validate (and all subclasses), 3 - adds a testcase
comment:5 Changed 12 years ago by
| Owner: | changed from nathan to Douglas Hays |
|---|---|
| Status: | assigned → new |
Changed 12 years ago by
| Attachment: | FormValidation.2.patch added |
|---|
Update to formValidationPatch - which contains a test case.
Changed 12 years ago by
| Attachment: | FormValidation.2.3.patch added |
|---|
Updated patch - fixes scrolling tooltip issue and moves test case into Form.html
comment:6 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Needs further discussion...