In webform.api.php, _webform_render_component() should use #theme_wrappers instead of #prefix and #suffix. This matches the components included with Webform.

CommentFileSizeAuthor
#2 webform-api_theme_wrappers-2620216-2-D7.patch730 bytesLiam Morland
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Liam Morland created an issue. See original summary.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Assigned: Liam Morland » Unassigned
Status: Active » Needs review
FileSize
webform-api_theme_wrappers-2620216-2-D7.patch730 bytes
quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch as a volunteer and at Backdrop CMS, Lullabot commented
Status: Needs review » Reviewed & tested by the community

Absolutely. Thanks again Liam!

  • DanChadwick committed aca0e75 on 7.x-4.x 
    Issue #2620216 by Liam Morland: Fixed _webform_render_component(), use #...
DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick at PreviousNext commented

Committed to 7x-4.x.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick at PreviousNext commented
Status: Reviewed & tested by the community » Fixed
DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick at PreviousNext commented
Version: 7.x-4.x-dev » 8.x-4.x-dev
Status: Fixed » Patch (to be ported)
DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick at PreviousNext commented
Category: Bug report » Task
fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented
Version: 8.x-4.x-dev » 7.x-4.x-dev
Status: Patch (to be ported) » Fixed

Closing to clear out the old Webform 8.x-4.x branch. See #2827845: [roadmap] YAML Form 8.x-1.x to Webform 8.x-5.x.

  • DanChadwick committed aca0e75 on 8.x-5.x 
    Issue #2620216 by Liam Morland: Fixed _webform_render_component(), use #...

Status: Fixed » Closed (fixed)

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