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.

CommentFileSizeAuthor
wrong_class_on_body_tag.jpg345.19 KBplamenut
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yannisc’s picture

Assigned: Unassigned » georgemastro
georgemastro’s picture

Status: Active » Closed (fixed)

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:

function 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';
  }
}