I'm trying to do an install profile that enables this theme instead of Garland. The problem is that by default, the Navigation menu ends up in the admin_header region, as pictured in the attached screenshot. This is because there is no 'left' region, only 'right'.

I'm not really sure what can be done about this, apart from renaming 'right' to 'left' and that just seems awfully confusing. :P

CommentFileSizeAuthor
Picture 5.png57.37 KBwebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Well, as a workaround I'm doing:

db_query("UPDATE {blocks} SET region = 'right' WHERE region = 'admin_header'");
cache_clear_all();

That seems fine. It'd be neat if this theme could somehow do this automatically when it's enabled, but that might not be possible.

Anonymous’s picture

Anonymous (not verified) commented
Assigned: Unassigned »

Yeah, I've noticed this when I enable the theme on a fresh install. I'm considering just rendering the content of the 'left' region in the right column above the 'right' content. I need to play around with it to see if I can have it render there without exposing that region as an option.

I don't really like the idea of updating someone's database when they enable the theme. It makes me feel dirty.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

I added a region with the internal name 'left' and the pretty name 'Upper Sidebar'. The 'right' region now has the pretty name 'Lower Sidebar'. Each, if present, is printed in the right visual sidebar with 'left' rendered above 'right'. This accommodates switching from a theme that has left region content.

This fix is addressed in:
nitobe.info v 1.4
page.tpl.php v 1.8

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Oh nice. This is a much smoother fix than my dirty hack. Thanks a lot!

Anonymous’s picture

Anonymous (not verified) commented

I think this will also make it easier adapting the theme for right-to-left languages whenever I get around to that.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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