Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I came across the issue that if you enter a field description and disable opacity input, the description appears twice in the field widget.
Also, I think wrapping the widget in a fieldset is not neccesary (and not beneficial) without opacity, it would only include a single field.
Comment | File | Size | Author |
---|---|---|---|
#12 | color-wrong.jpg | 11.62 KB | rwam |
#12 | color-correct.jpg | 17.73 KB | rwam |
#6 | double-field-description-3111039-6.patch | 523 bytes | balintk |
| |||
#3 | double-field-description-3111039-3.patch | 483 bytes | kmetz |
Comments
Comment #2
kmetz CreditAttribution: kmetz as a volunteer commentedComment #3
kmetz CreditAttribution: kmetz as a volunteer commentedComment #4
bartlangelaanComment #6
balintk CreditAttribution: balintk commentedI ran into the same issue. Thank you for the patch, @kmetz. Here is my slightly adjusted version which still renders the container element around the field so that JavaScript logic that expects that markup (i.e.
Drupal.behaviors.color_field_spectrum
) doesn't break.Comment #7
lonalorePatch from #6 works perfectly. Thank you!
Comment #8
DuneBLI confirm tha #6 is perfect!
Thank you
Comment #10
NickDickinsonWildeThanks for the patch and testing, committed.
Comment #12
rwam CreditAttribution: rwam commentedSorry, but this patch breaks the spectrum widget.
EXPECTED BEHAVIOR using
$element['#type'] = 'fieldset';
CURRENT BEHAVIOR using
$element['#type'] = 'container';
@NickDickinsonWilde - can this issue be reopened or should I create a new ticket?