Problem/Motivation

The component list in the webform settings (/admin/config/content/webform) was intended to be a fieldset. But it is not.
It would be better to still make it into a fieldset as the list is quite large and it would be nice to make it collapsible.

Proposed resolution

Fix the fieldset.
Just see the patch in #2, it clearly was a simple, hard to notice, mistake.

Remaining tasks

None.

User interface changes

The component list, which was always intended to be a fieldset, is now really a fieldset.
But the difference is marginal, and functionally there is no difference at all.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 webform-fix_settings_components_fieldset-3041384-2.patch508 bytespaulvandenburg

Comments

paulvandenburg created an issue. See original summary.

paulvandenburg’s picture

paulvandenburg commented
Status: Active » Needs review
StatusFileSize
new webform-fix_settings_components_fieldset-3041384-2.patch508 bytes

This patch fixes the lost fieldset.

johankleene’s picture

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

Reviewed and tested. Nice improvement.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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