- Issue created by @heykarthikwithu
- 🇮🇳India heykarthikwithu Bengaluru 🌍
I see we would need to add something like this within -
formatters/fieldset/fieldset.js
$('.fieldset__label').first().addClass('form-required');
- First commit to issue fork.
- Status changed to Needs review
28 days ago 9:53pm 31 May 2024 - 🇬🇧United Kingdom NexusNovaz
I've added the line you mentioned above to the file. Tested on a vanilla drupal 10 instance and the first fieldset is marked as required. Though, i suspect another change may be needed to require the second fieldset.
- 🇬🇧United Kingdom NexusNovaz
Apologies!
Updated the location and the line of code. I believe it now works as expected. Please could !55 this be reviewed
- Status changed to RTBC
19 days ago 7:14am 10 June 2024 - 🇦🇺Australia jannakha Brisbane!
this issue affects some themes, eg Claro or Olivero
MR!55 fixes the issue - tested on D 10.2.7 on Claro and Olivero.
before
after
Why do tests fail?
- 🇦🇺Australia jannakha Brisbane!
@Edde20
what's in the #8 patch?
does the issue require re-testing?
please set status of the issue to "needs review" and add some description to what's in the patch.