The region names have already been ported to sidebar_first and sidebar_right, but the css ids haven't been changed yet. This is minor, but important to support Drupal standards and make it easier to move to D8 in the future.

I do worry that Rubik (Tao most popular child theme) relies on Tao's page.tpl.php but it can copy an old version of page.tpl.php (bad) or upgrade its css selectors (good). It's up to Rubik's mantainers, nonetheless.

Comments

barraponto’s picture

Status: Active » Needs review
Issue tags: +Tao D7CX
StatusFileSize
new1.18 KB
coffeymachine’s picture

Status: Needs review » Fixed

There was a small issue with the patch getting 'sidebar_first' mixed up with 'sidebar-first'. I fixed that and went ahead and committed to 7.x-3.x.

This shouldn't affect Rubik as it does not make use of the sidebar_first or sidebar_second regions. Also there are no references to #left or #right in the Tao or Rubik CSS that need to be changed, so I don't see this causing any issues for Rubik or Tao.

Anyone else out there, however, that has built subthemes on Tao may need to update their CSS if they have any selectors on #left or #right. They would need to change them to #sidebar-first or #sidebar-second respectively.

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

  • Commit f06452b on 7.x-3.x, 8.x-3.x by CoffeyMachine:
    Issue #1423552 by barraponto, CoffeyMachine:  and  variables are now...