Comments on: How to test wp_nav_menu() with Thematic?

By: Martijn

Martijn — Mon, 09 Aug 2010 20:51:21 +0000

I would like to know in what file this code needs to be placed…

By: Kiper IT-konsult

Kiper IT-konsult — Fri, 25 Jun 2010 10:41:43 +0000

Thanks for all your hard work, Chris!

It’s great that you continue to develop Thematic for WP3. I have gotten so used to Thematic that it is hard to start on a new project without it…

By: Chris

Chris — Tue, 22 Jun 2010 14:39:49 +0000

Thanks a lot, Gene. Will change this later today and change Thematic.

By: Gene

Gene — Mon, 21 Jun 2010 15:31:42 +0000

Heads up in the final WP 3.0 things changed
`add_theme_support( ‘nav-menus’ );`

needs to be:

`add_theme_support( ‘menus’ );`

to work properly.

-Gene

By: Chris

Chris — Fri, 18 Jun 2010 18:33:35 +0000

Thanks a lot for your help, Theo. I’m currently testing Thematic against the final release of WordPress and doing some adjustments. Will post something later today.

Chris

By: Theo Ribeiro

Theo Ribeiro — Fri, 18 Jun 2010 17:49:27 +0000

Hi Chris, I was trying to implement the code above with the Final Release version of WP 3 and it didn't seem to be working. After some research, I solved the problem and posted the solution on the Thematic Forums I'm posting the code bellow, I don't know if it can be improved any further:


// We declare that our theme supports wp_nav_menu()
add_theme_support( 'nav-menus' );

// We Register the a new menu for the theme called "Primary Menu"
function register_primary_menu() {
	register_nav_menu( 'primary-menu', __( 'Primary Menu' ) );
}
add_action( 'init', 'register_primary_menu' );

// We remove the standard Thematic menu
function remove_menu() {
	remove_action('thematic_header','thematic_access',9);
}
add_action('init', 'remove_menu');

// We create the new wp_nav_menu called "Primary Menu" in our theme
function new_access() { ?>
	
		
			
			
			 'primary-menu', // we define this as being the previously registered "Primary Menu"    			
                          			'menu_class' => 'sf-menu', // we assign the sf-menu class to the menu ul so that superfish workds
                          			'container_class' => 'menu' // we assign the menu class to the menu container div so to keep it compatible with the Thematic menu styling
                          		));
			?>



By: Joe
Joe — Mon, 14 Jun 2010 17:40:42 +0000
actually, disregard my last comment… looks like the menu gets an ID based on the name specified in the menu name field…



By: Joe
Joe — Mon, 14 Jun 2010 17:35:27 +0000
i changed this line:
return preg_replace(‘//’, ”, $ulclass, 1);
to this:
return preg_replace(‘//’, ”, $ulclass, 1);
and the menu works…  looks like the new WP menu function adds an ID called “menu-main”
cheers!



By: Chris
Chris — Fri, 04 Jun 2010 12:48:23 +0000
Thanks a lot, Perry. I’ll test the current SVN copy.
Chris



By: Perry
Perry — Thu, 03 Jun 2010 11:04:44 +0000
Hi there – just thought I would let you know that I just tried this on WP3 RC 1: the menu appears OK but the filter to add the “sf-menu” class to the ul doesn’t seem to work.