If an accordion item has a child with an error, the accordion item should be open when the form reloads and displays errors. Otherwise, the user doesn't know which accordion item holds the error and might have to open other accordion items to find the one holding the error.

CommentFileSizeAuthor
#2 field_group-accordion-item-open-on-error-2923831-2.patch640 bytesoknate
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

oknate created an issue. See original summary.

oknate’s picture

oknate
Location Greater New York City Area
CreditAttribution: oknate as a volunteer and commented
FileSize
field_group-accordion-item-open-on-error-2923831-2.patch640 bytes

Here's a patch that works for me locally.

Now if I trigger an error on a child element, the accordion item is open when the form reloads.

oknate’s picture

oknate
Location Greater New York City Area
CreditAttribution: oknate as a volunteer and commented
Status: Active » Needs review
Amerie’s picture

Amerie CreditAttribution: Amerie commented

This works for me. Thanks!

Kris77’s picture

Kris77 CreditAttribution: Kris77 commented

#2 work for me too with 8.x-3.0-beta1.

Thanks @oknate

  • zuuperman committed 9ab1b99 on 8.x-3.x authored by oknate 
    Issue #2923831 by oknate: Accordion items with children with errors not...
nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop as a volunteer and at Wunder commented
Status: Needs review » Fixed

Thx for the patch. This is committed to dev

Status: Fixed » Closed (fixed)

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

Berliner-dupe’s picture

Berliner-dupe CreditAttribution: Berliner-dupe commented

Implements this feature also for the 7-version, plz.