Drupal 6 is rendering blocks in the correct regions, but bypassing the new regions.tpl.php. The effect is that sidebar classes are not printed, breaking layout.

Attached patch

CommentFileSizeAuthor
#3 sidebar.patch856 byteskdebaas
sidebar.patch864 byteskdebaas

Comments

kdebaas’s picture

kdebaas commented
Title: region.tpl.php / region-sidebar.tpl.php is not rendered » region.tpl.php is not rendered, breaking layout for sidebars
Status: Active » Needs review

Attached patch above does not solve this (I wouldn't even know if this is possible in drupal 6), but reverts page.tpl.php to its original functionality, while respecting the new css class naming.

kdebaas’s picture

kdebaas commented
Title: region.tpl.php is not rendered, breaking layout for sidebars » page.tpl.php does not print sidebar classes, and region-sidebar.tpl.php isn't rendered, breaking layout

Updated title to be more accurate.

kdebaas’s picture

kdebaas commented
StatusFileSize
new sidebar.patch856 bytes

Oops, forgot to attach the patch.

Herman Hiddema’s picture

Herman Hiddema commented

Do you have block translation enabled? If so, this may be the same bug as #644044: Activating module Block Translation breaks sidebar rendering

kdebaas’s picture

kdebaas commented
Title: page.tpl.php does not print sidebar classes, and region-sidebar.tpl.php isn't rendered, breaking layout » Region-sidebar.tpl.php isn't rendered when Block Translation is activated, breaking layout
Status: Needs review » Closed (duplicate)

You're absolutely right. In that case, the patch doesn't apply.