While working on getting two sidebars in, I found the current widths of the content + sidebar do not properly add up.

content 680
Margin-left 15
width 640
Padding-right 20
Margin-right 5

Sidebar-first 265
25
225
15

TOTAL 945

There is a spare 15px somewhere!

See also my git branch with support for two sidebars.

Comments

emmajane’s picture

emmajane CreditAttribution: emmajane commented
Version: » 7.x-1.x-dev

Is it a spare 15px or a "leave this space to make floats work" spare 15px?

Also: can you make the nice CVS patchy patches please? i.e. attach patch to this issue against current d.o CVS. I'm not sure how much what you've got differs from what I'm currently working against from the CVS repo.

ta!

emmajane’s picture

emmajane CreditAttribution: emmajane commented
Priority: Critical » Normal

Downgrading this issue to normal. Also: this issue is probably irrelevant once #713444: Add a second sidebar is fixed.

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

that one is fixed

jensimmons’s picture

jensimmons CreditAttribution: jensimmons commented
Status: Active » Fixed

This was fixed by #749062: Adjust grid to put more space between things. All done.

Status: Fixed » Closed (fixed)

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