The new Seven theme performs a CSS reset which removes all margin from the P tag, but then never reapplies any margin to the tag. This causes some of the help pages to be pretty hard to read as their is no space between paragraphs.

You can see this in action by going to a page like admin/help/simpletest, or checkout the attached screenshots.

Attached patch adds 10px margin to the bottom of the P element in Seven. 10px seems to be consistent with margin used on other elements. It also adds 10px bottom margin to LI tags on the admin/help page as they were demonstrating a similar spacing issue. See attached screenshots.

CommentFileSizeAuthor
eojthebrave-seven-p-li-spacing.patch423 byteseojthebrave
eojthebrave-testing-help-before.png52.28 KBeojthebrave
eojthebrave-testing-help-after.png74.27 KBeojthebrave
eojthebrave-help-main-before.png47.3 KBeojthebrave
eojthebrave-help-main-after.png45.8 KBeojthebrave

Comments

yoroy’s picture

yoroy commented
Priority: Minor » Normal
Status: Needs review » Closed (duplicate)

Very true. Lets fix this in #563390: Seven theme lacks formatting on common html elements such as lists, paragraphs & <code> which already has a working patch. thanks!