Not sure that this counts as a bug, since each site's layout is different, but...

A 3 column layout with the supplied files gives a 180-20-480-30-230 pixel format. The gaps between the two columns are different widths, which looks a bit odd to me.

The attached layout.css gives a 195-20-510-20-195 split across the three columns.

PS Thanks very much for releasing this theme. I liked the concept behind the Zen theme, but when I downloaded it I found the default view quite ugly, and I don't have the CSS skills to start from scratch. Your theme arrived at just the right time for me!

CommentFileSizeAuthor
#2 two-column-layout-fix.patch2.2 KBgeerlingguy
layout.css_.txt6.63 KBdavidhk

Comments

geerlingguy’s picture

geerlingguy commented

Thanks! Since I created the theme for a site specifically, then rooted out some of the custom CSS, things like this were overlooked for the first release. I'll get that first column to look a bit better, using what you've provided.

And I agree; it is best if Zen either (a) ships with a beautiful subtheme, or (b) ships as core alone, with the idea that it is solely a base theme. In fact, I think the latest releases do (a): #426750: Remove Zen Classic sub theme.

I'll update this issue when the patch is applied.

geerlingguy’s picture

geerlingguy commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new two-column-layout-fix.patch2.2 KB

I decided to go with a 220-500-220 layout, as that makes it simpler in my mind... I'm a fan of relatively wide sidebars, and users will still be able to customize the widths of their sidebars *relatively* easily ;-)

Patch attached; will apply to CVS tonight. Might roll a new version tonight, if the rest of my testing goes well.

geerlingguy’s picture

geerlingguy commented
Assigned: Unassigned » geerlingguy
Status: Reviewed & tested by the community » Fixed

Applied to CVS.

geerlingguy’s picture

geerlingguy commented
Status: Fixed » Closed (fixed)