Hi,

Problem/Motivation

Minimum length can be set in the component Form display tab but it is not added to the form item attributes (while maxlength is).

Steps to reproduce

Set minimum length to a text/textarea/email... component, check the HTML source code: no minlength attribute.

Proposed resolution

Same issue exists for Webform 8.x-5.x-dev (https://www.drupal.org/project/webform/issues/2843424) and the proposed patch might help as a starting point.

Thanks

CommentFileSizeAuthor
#7 beforePatch.png14 KBpaulocs
#7 afterPatch.jpg29.17 KBpaulocs
#5 3216513-4.patch7.42 KBjrockowitz
#2 3216513-2.patch1.76 KBjrockowitz

Comments

PhilY created an issue. See original summary.

jrockowitz’s picture

jrockowitz commented
Version: 6.0.3 » 8.x-5.x-dev
StatusFileSize
new 3216513-2.patch1.76 KB

This issue should also be addressed in 8.x-5.x

jrockowitz’s picture

jrockowitz commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 3216513-2.patch, failed testing. View results

jrockowitz’s picture

jrockowitz commented
StatusFileSize
new 3216513-4.patch7.42 KB
jrockowitz’s picture

jrockowitz commented
Status: Needs work » Needs review
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new afterPatch.jpg29.17 KB
new beforePatch.png14 KB

Tests look good and patch is working as expected.
Before patch no minlength attribute was set.

Moving to RTBC.

phily’s picture

phily
he/him
Primary language French
Location 🇪🇺🇫🇷 Paris, France
commented

Hi,
Patch #5 seems OK using Webform 6.0.3 and Drupal 8.9.16, thanks to jrockowitz for that quick fix.

jrockowitz’s picture

jrockowitz commented
Status: Reviewed & tested by the community » Fixed
Webform logo This ticket was completed using Open Collective funds.
Please consider joining the Webform module's Open Collective to help ensure that the Webform module is maintained.

  • jrockowitz authored 4d43725 on 8.x-5.x 
    Issue #3216513 by jrockowitz, paulocs: Support #minlength in components
  • jrockowitz authored 4d43725 on 8.x-5.x
    Issue #3216513 by jrockowitz, paulocs: Support #minlength in components

  • jrockowitz authored 4d43725 on 6.x 
    Issue #3216513 by jrockowitz, paulocs: Support #minlength in components
  • jrockowitz authored 4d43725 on 6.x
    Issue #3216513 by jrockowitz, paulocs: Support #minlength in components

Status: Fixed » Closed (fixed)

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