- Issue created by @g-brodiei
- ๐น๐ผTaiwan g-brodiei
g-brodiei โ changed the visibility of the branch 3440309-first-option-of to hidden.
- Merge request !456fix: issue-3440309 required first checkboxes option with wrapper type form element โ (Open) created by g-brodiei
- last update
over 1 year ago 536 pass - ๐น๐ผTaiwan g-brodiei
Created MR for 6.2.x
Providing another patch that applies against 6.2.2
Set to NR.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 4:48pm 11 April 2024 - last update
over 1 year ago Patch Failed to Apply - Status changed to RTBC
8 months ago 4:16pm 16 January 2025 - ๐ง๐ชBelgium thierry.beeckmans
We faced the same issue where the first checkbox was required when the checkboxes field with wrapper form element where configured as conditional fields.
The merge request applied successfully on branch 6.2.7 and the issue on the webform is solved by it.
- ๐บ๐ธUnited States hungdo
I am getting the error message keeps showing even checked other options, the error class is still appearing in the first option. Attaching a patch.
- ๐บ๐ธUnited States jason.ullstam Lexington, KY
jason.ullstam โ made their first commit to this issueโs fork.
- ๐บ๐ธUnited States jason.ullstam Lexington, KY
I attempted the patch from #5 and it failed. I also tested just making the change manually and I was still seeing the issue. Tried the patch from #9 and it ran successfully however did not resolve the issue for me. Running on core 10.4.2 with webform 6.2.9. I created a MR with a possible resolution. It did work on my instances. Also uploading a patch file for testing.
- Status changed to Needs review
13 days ago 7:21pm 3 September 2025 - ๐บ๐ธUnited States jrockowitz Brooklyn, NY
I think I found the root cause, the 'js-webform-type-*' class type needs to added to the form_element wrapper.
6.2.x is close to being unsupported. I switched the ticket to 6.3.x, but Iam stuck on how to create a 6.3.x MR.
Could someone help create an MR for 6.3.x using the attached patch.
The attached webform replicates the issue.
- Merge request !723[#3440309] feat: First option of required checkboxes/checkboxes other become... โ (Merged) created by jrockowitz
- ๐บ๐ธUnited States jrockowitz Brooklyn, NY
jrockowitz โ changed the visibility of the branch 3440309-fix-webform-checkbox-required to hidden.
- ๐บ๐ธUnited States jrockowitz Brooklyn, NY
jrockowitz โ changed the visibility of the branch 3440309-first-option-of to hidden.
- ๐บ๐ธUnited States jrockowitz Brooklyn, NY
jrockowitz โ changed the visibility of the branch 6.2.x to hidden.
- ๐บ๐ธUnited States jrockowitz Brooklyn, NY
I figured out how to create an MR for 6.3.x. I am expecting a few tests to fail because we are adding classes to form elements.
- ๐บ๐ธUnited States jrockowitz Brooklyn, NY
As expected below are the broken tests
- Drupal\Tests\webform\Functional\Element\WebformElementOtherTest
- Drupal\Tests\webform_example_element\Functional\WebformExampleElementTest
- Drupal\Tests\webform\Functional\Element\WebformElementCheckboxesTest
- Drupal\Tests\webform\Functional\Element\WebformElementCompositeTest
- Drupal\Tests\webform\Functional\Element\WebformElementEmailConfirmTest
- Drupal\Tests\webform\Functional\Element\WebformElementMultipleTest
- Drupal\Tests\webform\Functional\Element\WebformElementRadiosTest
- Drupal\Tests\webform\Functional\Element\WebformElementTableTest
- Drupal\Tests\webform\Functional\States\WebformStatesHiddenTest
- ๐บ๐ธUnited States jrockowitz Brooklyn, NY
If the tests pass. this could be RTBC.
-
jrockowitz โ
committed 392af8d1 on 6.3.x
[#3440309] feat: First option of required checkboxes/checkboxes other...
-
jrockowitz โ
committed 392af8d1 on 6.3.x
Now that this issue is closed, please review the contribution record.
As a contributor, attribute any organization helped you, or if you volunteered your own time.
Maintainers, please credit people who helped resolve this issue.
-
jrockowitz โ
committed 392af8d1 on 6.x
[#3440309] feat: First option of required checkboxes/checkboxes other...
-
jrockowitz โ
committed 392af8d1 on 6.x