Closed (fixed)
Project:
Panels Everywhere
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Jul 2011 at 00:30 UTC
Updated:
17 Sep 2011 at 20:51 UTC
I believe that the tpls in Panels Everywhere have not been upgraded properly to Drupal 7. This is causing a problem when I try to make a change to pane-navigation.tpl.php to add element-invisible to the H2 on main_menu.
Drupal 7 prints the menu:
<?php print theme('links__system_main_menu', array('links' => $main_menu, 'attributes' => array('id' => 'main-menu', 'class' => array('links', 'inline', 'clearfix')), 'heading' => t('Main menu'))); ?>
I usually amend that with:
<?php print theme('links__system_main_menu', array('links' => $main_menu, 'attributes' => array('id' => 'main-menu', 'class' => array('links', 'inline', 'clearfix')), 'heading' => array('text' => t('Main menu'), 'level' => 'h2', 'class' => array('element-invisible',),))); ?>
When I try to do that in pane-navigation.tpl.php the menu doesn't print at all.
Additionally, it looks like it's still using the old class="clear-block" when it should be class="clearfix". I'd be happy to write a patch for this and anything else that would help, but I'm not sure how to start fixing the $main_menu variable issue.
Comments
Comment #1
pontus_nilssonI can't find what you are quoting in the code.
pane-navigation.tpl.php doesn't contain the code you are referring to.
theme.inc builds the menus in the declaration of template_preprocess_pane_navigation.
The code there uses
class="clearfix".If you want to change how the menus are built you could implement template_process_pane_navigation in your own theme to do overrides.
Maybe I am missing something here :)
Comment #2
Letharion commentedThere are two stray clear-block in pane-navigation.tpl.php, I've fixed (but not pushed) those.
As for the general issue, it's quite possible that the template files haven't not gotten all the attention they need, but like Pontus, I don't understand what's you're trying to achieve.
Please be more elaborate on what you're doing and why, and we'll work it out from there.
Comment #3
Letharion commentedThe clear-blocks are fixed, I'm closing this one.
If there are other issues regarding D6 standards in D7 templates, this issue can be re-opened.
Otherwise please open a new issue.