Context Navigation

← Previous Ticket
Next Ticket →

#24732 closed enhancement (fixed)

HTML5 required property to append or replace "aria-required"

Reported by:	MMN-o	Owned by:	wonderboymusic
Priority:	normal	Milestone:	4.2
Component:	Comments	Version:	3.6
Severity:	normal	Keywords:	has-patch needs-refresh
Cc:		Focuses:

Description

I found it slightly annoying that my browser kept sending the comment forms on my blog even if I forgot to fill out name and email.

Fortunately HTML5 has a property for this, 'required', which can force form fields to be filled before a user's browser submits the form.

Patch is attached that adds HTML5 required statement to required fields in default comment_form function.

It may be an idea also to entirely remove the "aria-required" property, as it is not really standardised.

For what I know, Wordpress tends to have <!DOCTYPE html>, which means the property doesn't need to be " required='required'", but rather just " required".

Both HTML5 and XHTML versions are attached.

Attachments (4)

wp-xhtml-required.patch (3.0 KB ) - added by MMN-o 13 years ago.: XHTML patch for comment_form required fields
wp-html5-required.patch (3.0 KB ) - added by MMN-o 13 years ago.: HTML5 patch for comment_form required fields
wp-html5-required_for_comment_template.patch (3.0 KB ) - added by MMN-o 13 years ago.: HTML5 patch, comment-template.php
wp-xhtml-required_for_comment_template.patch (3.0 KB ) - added by MMN-o 13 years ago.: XHTML patch, comment-template.php