Posted by plamenut on October 30, 2012 at 8:05am
3 followers
Jump to:
| Project: | Open Deals Drupal Distribution |
| Version: | 7.x-1.9 |
| Component: | Miscellaneous |
| Category: | bug report |
| Priority: | normal |
| Assigned: | georgemastro |
| Status: | closed (fixed) |
Issue Summary
Hi,
Let me say first that your work is impressive: Thank you for sharing it with the community.
Don't know if it's only me but i think I've found an issue with the body class that is added to each page regarding the presence (or not) of the sidebars. Namely, even if i have a sidebar with blocks, the printed class on body tag is still "no-sidebars" (please, check the attachment) thus preventing me to use a full width for the content div like:
.no-sidebars #content {width:value;}
Didn't found a solution how to fix it yet but if i do find one, ill post it here.
Thanks.
| Attachment | Size |
|---|---|
| wrong_class_on_body_tag.jpg | 345.19 KB |
Comments
#1
#2
The fix was committed for the problem. If you want to correct this without updating existing installation of opendeals just add a template.php file to the theme's folder and add the following:
<?phpfunction opendeals_theme_preprocess_html(&$variables) {
// Add information about the number of sidebars.
if (!empty($variables['page']['sidebar_first']) && !empty($variables['page']['sidebar_second'])) {
$variables['classes_array'][] = 'two-sidebars';
}
elseif (!empty($variables['page']['sidebar'])) {
$variables['classes_array'][] = 'one-sidebar sidebar-first';
}
elseif (!empty($variables['page']['sidebar_second'])) {
$variables['classes_array'][] = 'one-sidebar sidebar-second';
}
else {
$variables['classes_array'][] = 'no-sidebars';
}
}
?>