Problem/Motivation

There's a minor style glitch in the modules table at /admin/modules. The Help module label does not align with the adjacent checkbox; it's 1em too low. Help is only module name affected.

Attached screenshot shows the appearance in webkit (Chromium 24), with the margin-top space highlighted.

The top margin comes from a style rule which is used on the /admin/help page. The new module table markup has @id attributes on all module labels, so the help page rule is being picked up on the modules table.

Proposed resolution

Remove the margin-top rule from Seven's style.css, or
change the id which is used on the help page

CommentFileSizeAuthor
#1 help-module-top-margin-1912112-1.patch287 bytesandrewmacpherson
help-margin-top.png36.12 KBandrewmacpherson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andrewmacpherson’s picture

andrewmacpherson CreditAttribution: andrewmacpherson commented
Issue tags: -authorize css white-space nowrap +CSS
FileSize
help-module-top-margin-1912112-1.patch287 bytes

This patch removes #help { margin-top: 1em; } from Seven's style.css
The effect on the help page layout is minor, and doesn't affect usability.

andrewmacpherson’s picture

andrewmacpherson CreditAttribution: andrewmacpherson commented
Status: Active » Needs review
dman’s picture

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

I can confirm that
* the issue exists in todays code, and does look silly.
* this patch resolves it effectively.
* there are no side effects.

Very minor, ready to go in.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

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