- 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
8 months 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
8 months 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. - Status changed to Needs review
6 months ago 1:13am 22 August 2024 - First commit to issue fork.
- 🇮🇳India dishakatariya
Hi, I have verified this issue, on the D11 version. This is working fine for me.
Testing steps:
1. Edit Article content type
2. Navigate to manage form display
3. Add nested field groups, move one mandatory field within the nested group
4. Add/Edit the article content
5. Mandatory Asterisk will be missing for Nested field groupsTesting Results:
Applied the patch successfully and aesterisk sign also working fine set as required fields.Attaching screeenshots.
Thanks
- 🇨🇴Colombia yasminOrj
Hi, I’ve reviewed the merge request from comment #13, and it works for me.
I’ve attached screenshots showing the before and after of checking the branch from the merge request in comment #13.
Before
After
Thanks