Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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!
Comment | File | Size | Author |
---|---|---|---|
#2 | two-column-layout-fix.patch | 2.2 KB | geerlingguy |
layout.css_.txt | 6.63 KB | davidhk |
Comments
Comment #1
geerlingguy CreditAttribution: geerlingguy commentedThanks! 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.
Comment #2
geerlingguy CreditAttribution: geerlingguy commentedI 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.
Comment #3
geerlingguy CreditAttribution: geerlingguy commentedApplied to CVS.
Comment #4
geerlingguy CreditAttribution: geerlingguy commented