Rather undescriptive text, but pretty much going from.

http://screencast.com/t/vyZRsVQr

To something like

http://screencast.com/t/IHfXqzWTu (dont mind the bottom spacing, and tiny blocks)

CommentFileSizeAuthor
#8 562284_decreasing_the_spacing_8.patch683 bytessign
#8 compact_mode_after.png6.66 KBsign
#8 compact_mode_before.png6.89 KBsign
#5 li-spacing.png8.29 KBBarisW
#4 seven_css.patch699 bytesBarisW

Comments

sreynen’s picture

sreynen commented
Status: Active » Fixed

A fresh d7 install no longer has these spacing issues.

Bojhan’s picture

Bojhan commented
Status: Fixed » Active

Wait, what? no that's impossible.

BarisW’s picture

BarisW commented
Assigned: Unassigned » BarisW

I think it would be best to descrease the margin on #block-system-main ul.admin-list li and increase the margin on div.admin-panel .description
By doing so, we only use the margin on the Module page where the descriptions are enabled.

I'll give it a shot!

BarisW’s picture

BarisW commented
Status: Active » Needs review
StatusFileSize
new seven_css.patch699 bytes

Changing padding and margin settings on ul.admin-list.compact is all that´s needed.

BarisW’s picture

BarisW commented
StatusFileSize
new li-spacing.png8.29 KB

And the screenshot

Bojhan’s picture

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

Good to go.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Needs review

I totally do not see the difference between before/after. Can someone help by posting a before/after screenshot with some arrows or something?

The "to something like" in the initial post looks absolutely hideous, so I'm really hoping that's not what we're shooting for. ;)

sign’s picture

sign commented
StatusFileSize
new compact_mode_before.png6.89 KB
new compact_mode_after.png6.66 KB
new 562284_decreasing_the_spacing_8.patch683 bytes

there is a colon at the end of one line instead of semicolon
note: this is for when compact mode is enabled (hidden description)

screens attached

Bojhan’s picture

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

Ok, back to RTBC. It was broken apparently, weird - when I tested it before it applied cleanly, maybe my HEAD was conflicting.

We are not shooting for whats in the screenshot in the initial post no :P, that was me being crazy with firebug

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -Usability, -D7UX

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

Issue tags: +Usability, +D7UX

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