...see attached screenshot. Tested with Bartik, Garland and Seven themes and it's the same case.

big space in the logo vtab

CommentFileSizeAuthor
themesettings_verticaltabs-space_in_the_logo_vtab.png30.48 KBklonos

Comments

andrewmacpherson’s picture

andrewmacpherson commented

I was scratching my head with this one, but I spotted the cause yesterday in Drupal core. There's an extra class on the logo fieldset, with an associated CSS clear: both. That's causing the settings to jump down past the bottom-most tab.

I'm not sure why it's there, could be some cruft left over from D6. I'll raise an issue with D7 core, cos it's be better to fix it there.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

...ok thanx. Please remember to post a link to that issue here when you find time to file it.

andrewmacpherson’s picture

andrewmacpherson commented
Assigned: Unassigned » andrewmacpherson
Status: Active » Postponed

Patch filed against Drupal core...
#1759090: Remove redundant CSS from theme settings form

Marking this issue postponed; we can close this when the core issue has been dealt with.

andrewmacpherson’s picture

andrewmacpherson commented
Status: Postponed » Fixed

After checking the backport policy for Drupal core, it seems that backports with markup/css changes will only be done for critical issues.

So I've added a workaround to this module. Easy enough to do in the form alter.

Fixed in dev, will be in 7.x-1.1
http://drupalcode.org/project/themesettings_extras.git/commit/3c9547f

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

Thanx for the fix Andrew. I wasn't aware of the policy either so thanx for that too.

Status: Fixed » Closed (fixed)

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

andrewmacpherson’s picture

andrewmacpherson commented
Assigned: andrewmacpherson » Unassigned
Issue summary: View changes