http://themeshaper.com/forums/ Help In Shaping WordPress Themes en Thu, 23 May 2013 11:55:02 +0000 http://themeshaper.com/forums/topic/add_theme_supportmenus#post-15162 Fri, 27 Aug 2010 22:29:34 +0000 squidz 15162@http://themeshaper.com/forums/ <p>Will do!! Thanks. </p> http://themeshaper.com/forums/topic/add_theme_supportmenus#post-14925 Mon, 23 Aug 2010 12:36:20 +0000 helgatheviking 14925@http://themeshaper.com/forums/ <p>squidz- my code definitely pre-dated this new announcement to not use add_theme_support. your post made me curious and both your snippets of code work for me on the latest, stable thematic (709) WITHOUT add_theme_support('menus'). give it a shot. </p> http://themeshaper.com/forums/topic/add_theme_supportmenus#post-14913 Sun, 22 Aug 2010 21:05:03 +0000 squidz 14913@http://themeshaper.com/forums/ <p>Chris - I'm new to Thematic and coming from designer-world. In experimenting with building a child theme for my own site, I'm interested in setting the main nav in a vertical fashion.</p> <p>I found a post that shows an easy snippet to try this. However, I've also found various links by Gene to your post here about not using add_theme_support('menus'), but calling register_nav_menu(s) instead. Another example by Helgatheviking used both. </p> <p>So, being somewhat clueless here, I'm curious what the correct way to use this code from <a href="(http://themeshaper.com/forums/topic/going-vertical-with-wp3-menus-enabled#post-13970)">Tarpontech</a> should be:</p> <p>function go_vertical( $args ) {<br /> $args[ 'menu_class' ] = 'sf-menu sf-vertical';<br /> return $args;<br /> }<br /> add_filter( 'thematic_nav_menu_args', 'go_vertical' );</p> <p>Using this in conjunction with Gene's:</p> <p>add_theme_support( 'menus' ); **** can register_nav_menu(s) directly replace this? ***</p> <p>function child_access_menu() {<br /> $menu_sys = 'wp_nav_menu';<br /> return $menu_sys;<br /> }<br /> add_filter('thematic_menu_type', 'child_access_menu'); </p> http://themeshaper.com/forums/topic/add_theme_supportmenus#post-14844 Thu, 19 Aug 2010 18:02:52 +0000 Chris 14844@http://themeshaper.com/forums/ <p>Hi,</p> <p>do not use add_theme_support('menus') in your functions.php. I already removed it from the current SVN copy.</p> <p>Here's the answer from Andrew Nacin regarding the question if a theme needs this call in the functions.php:</p> <blockquote><p> This is incorrect. add_theme_support('menus') is an internal process only. *It should never be directly called.* Instead, the theme should be calling register_nav_menu(s) which will trigger add_theme_support('menus') internally.</p> <p>Likewise, add_theme_support('widgets') is internal only. The theme should only be calling register_sidebar(s). </p></blockquote> <p>Chris </p>