Thematic Revision 652

ADDED:
- New filter thematic_page_menu_args to filter the arguments for wp_page_menu().
FIXED:
- The UL Class sf-menu will be created only for the wp_page_menu() in the header.

This entry was posted in Added, Fixed and tagged , , , . Bookmark the permalink. Post a comment or leave a trackback: Trackback URL.

3 Comments

  1. Posted April 5, 2010 at 15:27 | Permalink

    @Chris: Sorry to be a pain. I’ve just loaded r652 (I think?) to the test install, but for some reason, the UL class for sf-menu isn’t loading up for me, so it’s just a plain

    Test site: http://thematic.wpsnippets.com/

    • Posted April 5, 2010 at 15:39 | Permalink

      Do you use wp_page_menu() or wp_nav_menu()?

      • Posted April 5, 2010 at 15:51 | Permalink

        @Chris: Thanks for the reply.

        Apologies. I can see where my error is now: I added a ‘Home’ link to the menu by taking out the ‘comments //” from the sample child theme functions.php so this was used.

        function childtheme_menu_args($args) {
        $args = array(
        'show_home' => 'Home',
        'sort_column' => 'menu_order',
        'menu_class' => 'menu',
        'echo' => true
        );
        return $args;
        }
        add_filter('wp_page_menu_args','childtheme_menu_args');

        Once I take it out, the menu works just fine.

Post a Comment

Your email is never published nor shared. Required fields are marked *

*
*

You may use these HTML tags and attributes: <a href="" title=""> <abbr title=""> <acronym title=""> <b> <blockquote cite=""> <cite> <code> <del datetime=""> <em> <i> <q cite=""> <strike> <strong>

Latest via Twitter

@alexjvasquez Let me know which posts should be removed.