This only happens on the celju theme, as far as I my tests with other themes. Using Firebug, I couldn't figure out what styles are causing the issue. Can anyone help?

See error at: http://gummiogolof.vitum.net/tagadelic/chunk/2

CommentFileSizeAuthor
#1 wrapper-fix.zip4.67 KBjwolf
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jwolf’s picture

FileSize
4.67 KB

The problem is that tagadelic is using a wrapper class <div class="wrapper tagadelic">

This theme is also using a wrapper class to set the width of the various content areas.

One thing you can do to fix this is to change the theme's wrapper class to content-wrapper in both page.tpl.php and style.css. Attached are the modified page.tpl.php and style.css, zipped up, which *should* fix this. Just replace the theme's w/ these attached.

Personally, I think it's a mistake of the tagadelic maintainer(s) to use such a generic class. But then again, maybe it's my mistake to use such a generic class :P

jwolf’s picture

Component: Usability » Code
Assigned: Unassigned » jwolf
Status: Active » Needs review
gjk’s picture

Status: Needs review » Fixed

Thanks a bunch. I just figured this out myself a couple of minutes ago. It is indeed the <div class="wrapper tagadelic"> that is causing the problem. I agree that the tagadelic module (and modules in general) should refrain from using generic css class names. I fixed this in my css by adding:

.wrapper .tagadelic
{
    width: 650px;
}

Again, thanks for the quick reply. The issue is solved for me. And thanks a bunch for a great theme.

Do you have any hints for changing colors and adding images to the header or should that be a fairly straightforward css/html thingy?

jwolf’s picture

If you want to go the easy route, that will work too. :P

Status: Fixed » Closed (fixed)

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