Problem/Motivation

Currently it is not possible to add a description to fieldsets created by Field Group because the option is simply not available in the settings form.

Proposed resolution

Add the option to enter a description to the settings form of the "Fieldset" formatter.

Front end view

I'd suggest rerolling the patch without the ID.

CommentFileSizeAuthor
#5 enable_description_for-2794055-5.patch702 byteswelly
#5 Screenshot 2016-10-01 22.31.40.png52.56 KBwelly
#5 Screenshot 2016-10-01 22.31.55.png120.49 KBwelly
#2 field_group-enable_fieldset_description-2794055-2.patch1.01 KBstBorchert
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stBorchert created an issue. See original summary.

stBorchert’s picture

stBorchert
he/him
Primary language German
CreditAttribution: stBorchert at undpaul commented
Status: Active » Needs review
FileSize
field_group-enable_fieldset_description-2794055-2.patch1.01 KB

Simple patch ...

nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop as a volunteer and at Wunder commented

Ok to add the description. But why is there an id generated? We removed that because a lot of people didn't want ids. That's why that setting was added.

stBorchert’s picture

stBorchert
he/him
Primary language German
CreditAttribution: stBorchert at undpaul commented

Without the ID I got an error on render so I added the fallback.

welly’s picture

welly CreditAttribution: welly at PreviousNext commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
FileSize
Screenshot 2016-10-01 22.31.55.png120.49 KB
Screenshot 2016-10-01 22.31.40.png52.56 KB
enable_description_for-2794055-5.patch702 bytes

I've tested this patch and not able to reproduce the error that occurs without the ID. I removed the ID from the field and it works as described. Rerolled the patch with the id removed.

  • zuuperman committed 427f8cb on 8.x-1.x authored by welly 
    Issue #2794055 by welly, stBorchert: Enable description for fieldsets
nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop as a volunteer and at Wunder commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Thx for the new patch. I committed it to dev.

Status: Fixed » Closed (fixed)

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