Opened 14 years ago
Closed 14 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 14 years ago by
Cc: | bill added |
---|---|
Owner: | set to Douglas Hays |
comment:2 Changed 14 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 14 years ago by
Cc: | nathan added; [email protected]… removed |
---|---|
Status: | new → assigned |
comment:4 Changed 14 years ago by
Cc: | nathan removed |
---|---|
Reporter: | changed from guest to nathan |
Changed 14 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 14 years ago by
Owner: | changed from nathan to Douglas Hays |
---|---|
Status: | assigned → new |
Changed 14 years ago by
Attachment: | FormValidation.2.patch added |
---|
Update to formValidationPatch - which contains a test case.
Changed 14 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 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Needs further discussion...