Contrib.social

Allow hiding fieldset and title

Open on Drupal.org →
Created on 2 February 2019, over 5 years ago
Updated 9 December 2023, 10 months ago

The fact that complex IEFs are rendered in a table cell means the additional fieldset wrapper seems completely superfluous and often visually annoying. It can be easily overridden using the CSS rule display: contents, however in most usecases it will be unwished for, so it should go.

Even for simple IEFs a fieldset isn't necessarily wanted. To provide a seamless integration of the IEF, sitebuilders may not want any visual separation. Or they might want to put the IEF in a vertical tab instead. The only difference is that in the basic usecase a fieldset helps understanding how the IEF works.

Initially I didn't want to add yet another setting, being rather in favor of referring sitebuilders to the Field group module → that allows all kinds of fancy fieldsets, vertical tabs etc. However, it would be a bit irritating to force it for simple IEFs (though sitebuilder might want to get rid of it), yet do away with it for complex IEFs. Also, we're in RC phase, so we don't want to remove any functionality.

Therefore I am adding another setting, though using #states to expose only relevant settings. Default for simple IEFs would be "on", default for complex ones "off."

✨ Feature request
Status

Needs work

Version

3.0

Component

User interface

Created by

Pancho UTC+2 🇪🇺 EU

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024