I am currently getting these errors:

Notice: Undefined variable: main_group_width in include() (line 57 of /sw/var/www/d7-dev/sites/all/themes/abstract/templates/page.tpl.php).
Notice: Undefined index: sidebar_first in fusion_core_grid_info() (line 342 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_grid_info() (line 344 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_first in fusion_core_grid_info() (line 342 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_grid_info() (line 342 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_first in fusion_core_preprocess_page() (line 79 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_preprocess_page() (line 79 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).

Right now the content and right sidebars have full width. Fusion its still in dev snapshot and there is no usable Skinr for D7, so this issue might have to wait a little.

CommentFileSizeAuthor
#11 abstract-drupal-7-upgrade-1121678-9.patch1.56 KBmmenavas
#5 0001-Issue-1121678-by-ipwa-mmenavas-sidebar-widths-Update.patch8.18 KBmmenavas

Comments

sbsimmons86’s picture

sbsimmons86 commented

Subscribing...Im having the same issue.

ipwa’s picture

ipwa commented

Are you getting all these errors??

With the latest dev snapshot you should only be getting:

Notice: Undefined variable: main_group_width in include() (line 57 of /sw/var/www/d7-dev/sites/all/themes/abstract/templates/page.tpl.php).

Make sure you clear your cache.

sbsimmons86’s picture

sbsimmons86 commented

Yeah, Im only getting the one error now.

ipwa’s picture

ipwa commented

Awesome thanks. As I said before Fusion is still dev snapshot, so I'll try to fix this bug, but there's no rush until there's at least a fusion beta.

mmenavas’s picture

mmenavas commented
StatusFileSize
new 0001-Issue-1121678-by-ipwa-mmenavas-sidebar-widths-Update.patch8.18 KB

This patch does not solve this issue entirely, but it corrects the page.tpl.php file to render all regions (except for 'Content to') correctly. Stylesheets and div classes might have to be updated

ipwa’s picture

ipwa commented

Committed: http://drupal.org/commitlog/commit/12322/33f7151a27a5bec5845d7778dc31a95...

Thanks :)

ipwa’s picture

ipwa commented
Status: Active » Needs work
ipwa’s picture

ipwa commented
Title: sidebar widths (Fusion 7.x) » Drupal 7 upgrade
ipwa’s picture

ipwa commented

Just committed: http://drupal.org/commitlog/commit/12322/732bf82947171dc01ed90f743bd4bbf...

Now we have unstable1 for Drupal 7: http://drupal.org/node/1165662

Didn't want to do an alpha just yet because there are a couple of things to sort out like the 0 that comes from the node.tpl.php probably.

Main menus don't work in Fusion like regular themes btw, you need to assign your main menu block to the main menu region.

@mmenavas if you're going to continue helping out with patches, remember to git pull so you get my latest changes locally, thanks so much for the help.

mmenavas’s picture

mmenavas commented

No problem! I'm glad to help. I'll pick this up again in future code sprints.

mmenavas’s picture

mmenavas commented
StatusFileSize
new abstract-drupal-7-upgrade-1121678-9.patch1.56 KB

This patch cleans up part of node.tpl.php. I think abstract.info and page.tpl.php need to be updated to remove one of the sidebars as in Abstract 6.x-1x.

ipwa’s picture

ipwa commented
Issue summary: View changes
Status: Needs work » Fixed

Please use the 7.x-2.x branch now. Implemented with Bootstrap theme as parent theme.

Status: Fixed » Closed (fixed)

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