Problem/Motivation

Checkboxes don't support #states required

Proposed resolution

Refactor the webform's support for radio buttons #states required to also support checkboxes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jrockowitz created an issue. See original summary.

  • jrockowitz committed d41a25f on 2938414-states-checkboxes
    Issue #2938414: Checkboxes don't support #states required
    

  • jrockowitz committed 2667916 on 2938414-states-checkboxes
    Issue #2938414: Checkboxes don't support #states required
    

  • jrockowitz committed 758d31d on 2938414-states-checkboxes
    Issue #2938414: Checkboxes don't support #states required
    
jrockowitz’s picture

Status: Active » Needs review
FileSize
10.92 KB

  • jrockowitz committed 64bfc7e on 2938414-states-checkboxes
    Issue #2938414: Checkboxes don't support #states required
    
jrockowitz’s picture

The attached fixes a minor bug in the JS.

Maheshwaran.j’s picture

Assigned: Unassigned » Maheshwaran.j
Maheshwaran.j’s picture

Assigned: Maheshwaran.j » Unassigned
Status: Needs review » Reviewed & tested by the community
FileSize
128.4 KB

@jrockowitz

I tested in Drupal version 8.4.4.
Your patch applies cleanly and checkbox fields do support required fields now.

  • jrockowitz committed aba38c7 on 2938414-states-checkboxes
    Issue #2938414: Checkboxes don't support #states required
    

  • jrockowitz committed 8712de4 on 8.x-5.x
    Issue #2938414 by jrockowitz, Maheshwaran.j: Checkboxes don't support #...
jrockowitz’s picture

Status: Reviewed & tested by the community » Fixed

@maheshwaranj Thanks for the review.

Status: Fixed » Closed (fixed)

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