At the moment the width of sidebars is subtracted from the width of content, even if sidebars aren't being displayed. It would be great to be able to allow the main content to take up 100% width if sidebars are absent.

Comments

himerus’s picture

Assigned: Unassigned » himerus
Category: feature » bug

I will verify this, but can you send me screenshots of the settings in your theme settings page for content zone, and also a screen of the site without sidebars where the content is NOT full width?

By default, it "should" take the full container width for the content zone, and actually "ignore" the selected item for the content zone, so if neither sidebar is present, it should be taking the full width...

If this is failing, it's a bug, and not a feature request, so I'll update the ticket accordingly, and do some testing on my localhost tomorrow on trying to replicate.

himerus’s picture

Priority: Normal » Critical

Okay, I do see the issue here... replicated it in my sandbox...
Definitely a bug... will get fixed asap!!

Marking critical as well... this really IS quite important in the way the grid and usage of the ns() function should be working.

himerus’s picture

Status: Active » Fixed

I found the error in template.php under _omega_dynamic_widths

function _omega_dynamic_widths($width, $conditions, $vars) {
  foreach($conditions AS $variable => $zone) {
    if((isset($vars['page'][$variable]))) {
      $width = $width - $zone['width'];
    }
  }
  return $width;
}

changed to:

function _omega_dynamic_widths($width, $conditions, $vars) {
  foreach($conditions AS $variable => $zone) {
    if((isset($vars['page'][$variable])) && count($vars['page'][$variable]) > 0) {
      $width = $width - $zone['width'];
    }
  }
  return $width;
}

Change committed and fixed in Alpha8 release.

patrickharris’s picture

Priority: Critical » Normal
Status: Fixed » Active

Thanks!

himerus’s picture

Status: Active » Fixed

Marking back as fixed.

Let me know if there are any further issues with this. I plan to release the first Beta version for D7 this weekend. Most everything seems ironed out, and ready to put in production on D7.

Status: Fixed » Closed (fixed)

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

hexabinaer’s picture

Status: Closed (fixed) » Active

Erm - seems like I'm having the same issue as described above with 7.x-3.0
My fault? Anyone else?

superstar’s picture

Status: Active » Closed (fixed)

This is only occurring in 7.x-3.0 if debug mode is on. Log another issue.

code-brighton’s picture

This is still an issue... I wouldn't have found the solution if it hadn't been for this post... was another issue opened in the end? Be worth getting this fixed if possible

giupenni’s picture

In 7.x-3.1 the bug persist.

giupenni’s picture

Status: Closed (fixed) » Needs work

This is occurring if debug mode is off too.

giupenni’s picture

This is occurring if debug mode is off too.

zhangx1a0’s picture

I am having the same issue here. Debug is off.

Found the problem. I need to set content region as main region in content zone, then it will scale to full width.

jimmyr’s picture

I am still having a problem with this.

I'm not sure how to

'set content region as main region in content zone' as mentioned in the previous post.

steinmb’s picture

Issue summary: View changes
Status: Needs work » Closed (fixed)