This CSS declaration seems missing for fieldset description's. The below makes sure that top spacing is correct if a fieldset description is shown on top of a vertical tab.

div.vertical-tabs .vertical-tabs-panes fieldset.vertical-tabs-pane div.description {
  margin-bottom: 1em;
  margin-top: 1em;
}
CommentFileSizeAuthor
#3 vertitcal_tabs_without_fix.png15.98 KBhass
#3 vertitcal_tabs_after_fix.png12.28 KBhass
#1 vertical_tabs_missing_description_margin-D6.patch655 byteshass

Comments

hass’s picture

hass commented
Status: Active » Needs review
StatusFileSize
new vertical_tabs_missing_description_margin-D6.patch655 bytes

Patch attached with removed DIV

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Can you post screenshots before and after please?

hass’s picture

hass commented
StatusFileSize
new vertitcal_tabs_after_fix.png12.28 KB
new vertitcal_tabs_without_fix.png15.98 KB

See attached

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Project: Vertical Tabs » Drupal core
Version: 6.x-1.x-dev » 7.x-dev
Component: User interface » Garland theme
Issue tags: +vertical tabs

Just synced the D7 CSS with the backport module's CSS, so I think we need to bring this to core first then backport.

cosmicdreams’s picture

cosmicdreams commented

Since the creation of this issue, I think the margins have been improved. I vote to mark this issue as fixed.

cosmicdreams’s picture

cosmicdreams commented
Status: Needs review » Fixed

Double-checked the formatting, while this specific approach has not bee followed, margins are applied to fieldsets that make the formatting of this element better.

Status: Fixed » Closed (fixed)
Issue tags: -vertical tabs

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

Issue tags: +vertical tabs

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.