Problem/Motivation

Webform provides a custom form settings section where you can enter custom settings for your webform element.

These are surfaced in $variables['element']['<property_name>'] - it would be great if we linked CivicTheme design tokens via these settings so:

Webform advanced settings

Can be linked in `civictheme_preprocess_form_element__civictheme_field` via a helper say `_civictheme_preprocess_form_element__webform__overrides` - which only preprocesses webform elements and only links up known design tokens.

CommentFileSizeAuthor
webform-advanced-settings.png52.72 KBrichardgaunt

Comments

richardgaunt created an issue. See original summary.

richardgaunt’s picture

Issue summary: View changes
richardgaunt’s picture

Assigned: richardgaunt » alan.cole
Status: Active » Needs review
richardgaunt’s picture

Assigned: alan.cole » danielgry

When adding a container webform element in webform build settings.

If you populate the Custom settings with:
- component_theme: light | dark
- is_contained: true |false
- vertical_spacing: none | top | bottom | both
- with_background: true | false
- modifer_class:

You can control the layout / styles of the container from the UI. Useful for minor layout adjustments in webforms.

@danielgry ready for test

  • 716436ac committed on 1.x
    #3581086 Added link between webform custom settings and civictheme form...
richardgaunt’s picture

Assigned: danielgry » richardgaunt
Status: Needs review » Reviewed & tested by the community
richardgaunt’s picture

Version: 1.x-dev » 1.13.0
Assigned: richardgaunt » Unassigned
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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