Add a label to html-element wrappers [#2173351]

Hi,

for my current project, i need to display labels above html element field groups. With this functionality, its very easy to group the output of fields together and display them as "one field" while keeping the full power of display suite intact. (In contrast to a solution with tokens)

Patch is attached, its now possible to display labels and the label element is also configureable.

Best regards,
axe312

Comment	File	Size	Author
	html-wrapper-labels.patch.patch	4.28 KB	axe312

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

yannickoo

Berlin

CreditAttribution: yannickoo commented 14 January 2014 at 15:56

Status:

Needs review

» Reviewed & tested by the community

Patch looks good and works like the label functionality in other field group types. I think that the killed two whitespaces are totally okay.

Next time you should give your patch a better name like [project_name]-[short-description]-[issue-number]-[comment-number].patch.

Thank you for the patch :)

Comment #2

nils.destoop CreditAttribution: nils.destoop commented 13 March 2014 at 21:19

Thx for the patch. This was committed to dev. However, i changed the label markup a bit so it's equal to the div group type

Comment #3

nils.destoop CreditAttribution: nils.destoop commented 13 March 2014 at 21:19

Status:

Reviewed & tested by the community

» Fixed

Comment #4

27 March 2014 at 21:21

Status:

Fixed

» Closed (fixed)

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

Comment #5

15 June 2014 at 19:54

Commit ebefab2 on 7.x-1.x, 8.x-1.x authored by axe312, committed by zuuperman:
```
Issue #2173351 by axe312: Add a label to html-element wrappers.
```