Opened 10 years ago
Closed 10 years ago
#23345 closed New feature (wontfix)
Better message for required fileds in form validation
Reported by: | codefisher | Owned by: | nobody |
---|---|---|---|
Component: | Forms | Version: | 1.6 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
When a required field fails validation is says "This field is required". I think it should say something more like "Subject is required", assuming the label of the field is "Subject". It in my opinion is more user friendly.
I know this can be done by changing the error_messages argument/variable. But that means doing it for possibly hundreds of fields.
I will attach a patch, hope I get it right, it will be the first time ever I have posted a patch.
Attachments (1)
Change History (2)
by , 10 years ago
comment:1 by , 10 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Thanks for the suggestion, but there are backwards compatibility concerns. Everyone using
error_messages['required']
key would need to start passing'label'
inparams
. I don't think the gains outweigh the drawbacks.