Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#4020 closed defect (bug) (fixed)

Required fields must be marked as asterisk

Reported by: ramizmanked's profile ramizmanked Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: Showcase Keywords:


Field(s) must be marked by asterisk(*) symbol if it is required field on Submit a Site Form.

Also, we can apply client-side (i.e. javascript) validations to avoid sending blank form.

Attachments (1)

required.png (108.1 KB) - added by ramizmanked 5 years ago.

Download all attachments as: .zip

Change History (3)

5 years ago

#1 @dd32
5 years ago

  • Resolution set to maybelater
  • Status changed from new to closed

I'm okay with how this currently is.

The note about all fields being required is present above the submit button, and as all fields are required it doesn't really need to specify which fields are required.

Adding a client-side JS validation isn't out of the question, but the benefit in doing so is minimal. When the Showcase gets a redesign or make over, changing how this form works then would be good IMHO.

#2 @dd32
5 years ago

  • Owner set to dd32
  • Resolution changed from maybelater to fixed

In 8010:

Showcase: Add HTML5 required attributes to the Site Submit form to get some basic client-side validation prior to submit.

This also marks the email and url fields as their HTML5 type to get the basic content validation too.

Fixes #4020.

Note: See TracTickets for help on using tickets.