Following up on this comment in #536236: Custom CSS classes for form components, wrapper CSS classes option is not available anymore in markup component.

CommentFileSizeAuthor
#5 3028843-05.patch443 byteslucassc
#2 webform-wrapper_css_classes-3028843-2.patch439 bytestostinni

Comments

tostinni created an issue. See original summary.

tostinni’s picture

tostinni commented
Status: Active » Needs review
StatusFileSize
new webform-wrapper_css_classes-3028843-2.patch439 bytes

Here is a patch to add this option back.

tkcent’s picture

tkcent commented
Status: Needs review » Reviewed & tested by the community

Patch works as expected. This is very helpful when using CSS frameworks with grid system.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Is there a reason the classes couldn't just be added in the markup?

lucassc’s picture

lucassc
he/him
Primary language Portuguese, Brazil
Location Rio de Janeiro
commented
Status: Needs work » Needs review
Issue tags: -Needs reroll
StatusFileSize
new 3028843-05.patch443 bytes

Patch #2 rerolled. I'm not attaching the interdiff because there are no differences between one patch and another, but it's only changing 1 line, so you can easily compare with the first patch.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

People are able to add any classes they want to the markup they add to the page. It has been this way for a decade. Is there a reason this does not work?

lucassc’s picture

lucassc
he/him
Primary language Portuguese, Brazil
Location Rio de Janeiro
commented
Status: Needs review » Needs work

Hi, Liam!

Could you please provide some documentation or steps to reproduce?

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

The markup component just puts HTML on the page. Include any needed classes in that HTML.

ravi kant’s picture

ravi kant commented
Status: Needs work » Needs review

The custom wrapper class can add from Webform->Setting->form and add custom class on form attributes class.
This class will add on form element.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Needs review » Closed (outdated)

Drupal 7 is no longer supported. If this applies to a supported version, please re-open.

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

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

Maintainers, please credit people who helped resolve this issue.