Love this module!

I have a request for a minor change. When the module adds the required attribute to required form fields, it sets it to “true” which is incorrect. According to W3C guidelines:

“The presence of a boolean attribute on an element represents the true value, and the absence of the attribute represents the false value. The values “true” and “false” are not allowed on boolean attributes. To represent a false value, the attribute has to be omitted altogether.”

This means the value for the required attribute either shouldn’t be set or if it is, it’s set to “required” instead of “true”.

I'll attach a patch below.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

micnap created an issue. See original summary.

micnap’s picture

kim.pepper’s picture

Status: Active » Reviewed & tested by the community

LGTM

  • kim.pepper committed 685edbd on 7.x-1.x authored by micnap
    Issue #2821369 by micnap: Change value of required attribute from "true...
kim.pepper’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.