Problem/Motivation

When using Webform 6.2.9 and Client Side Validation 4.1.0 checkbox questions (checkbox and checkbox other), if I leave the question blank, it forces me to check the first checkbox in order to continue. If I select any checkbox other than the first, it continues to flag the question as an error. See video:

https://www.youtube.com/watch?v=6VFeFVhgBdc

Steps to reproduce

1. Install and configure webform
2. Install and configure client side validation for webform.
3. Make question that contains checkboxes OR checkboxes other.
4. Leave the checkbox or checkbox other question blank.
5. Select every option other than the first.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork webform-3500627

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

albertho425 created an issue. See original summary.

girish_lnweb’s picture

Assigned: Unassigned » girish_lnweb
cilefen’s picture

Status: Active » Postponed (maintainer needs more info)

Is this bug reproducible with Webform alone?

albertho425’s picture

When webform alone is used with checkbox or checkbox other, it says "Please check this box if you want to proceed" referring to the first checkbox. However, if I select any box, I can continue.

girish_lnweb’s picture

Assigned: girish_lnweb » Unassigned
jrockowitz’s picture

Version: 6.2.9 » 6.3.x-dev
jrockowitz’s picture

The error message positioning issue I can't reproduce. It might be custom theme and element settings related.

Screen shot

Attached is my test webform.

I improved the client-side validation to remove the red border from the unchecked checkbox.

jrockowitz’s picture

Status: Postponed (maintainer needs more info) » Needs review

jrockowitz’s picture

Status: Needs review » Reviewed & tested by the community

This feels very safe and addresses an issue that has been around for a while.

liam morland made their first commit to this issue’s fork.

  • jrockowitz committed 53452a53 on 6.3.x
    Issue #3500627 by jrockowitz, albertho425, girish_lnweb, cilefen: Drupal...
jrockowitz’s picture

Status: Reviewed & tested by the community » Fixed

  • jrockowitz committed 53452a53 on 6.x
    Issue #3500627 by jrockowitz, albertho425, girish_lnweb, cilefen: Drupal...

Status: Fixed » Closed (fixed)

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