ThemeShaper Forums Tag: thematic

ThemeShaper Forums Tag: thematic_access http://themeshaper.com/forums/ Help In Shaping WordPress Themes en Sat, 18 May 2013 19:22:08 +0000 colfelt on "Thematic menus - wrapping the <li> in the <a>" http://themeshaper.com/forums/topic/thematic-menus-wrapping-the-ltligt-in-the-ltagt#post-26701 Sun, 29 Apr 2012 00:11:14 +0000 colfelt 26701@http://themeshaper.com/forums/ It's all on my local server right now. Nevermind, your suggestion was WAY better. As usual, I was looking for a complicated way to solve a simple problem. Displaying the A element as a block and giving it a width attribute is all that was required. Thanks again HtV! helgatheviking on "Thematic menus - wrapping the <li> in the <a>" http://themeshaper.com/forums/topic/thematic-menus-wrapping-the-ltligt-in-the-ltagt#post-26697 Sat, 28 Apr 2012 17:54:35 +0000 helgatheviking 26697@http://themeshaper.com/forums/ do you have a link to the styling you're using / trying to achieve? i think it is a mistake to invert the markup... you're probably better off wrapping the text in an extra span so <code><li><a></code> which should also be easier to create via a Walker. but maybe you just need to put display: block on the A element and then add your padding to that. colfelt on "Thematic menus - wrapping the <li> in the <a>" http://themeshaper.com/forums/topic/thematic-menus-wrapping-the-ltligt-in-the-ltagt#post-26694 Sat, 28 Apr 2012 13:55:40 +0000 colfelt 26694@http://themeshaper.com/forums/ I read through ALL 5 pages of posts in Helga's epic Thematic Menus Demystified topic and the other topics I could find. But I *think* I have a new challenge for this forum. Is there a "clever" way to flip the order of the li and a tags output in wp_nav_menu? I know I could remove thematic_access and smash in my own hand-rolled menu in it's place. But that seems like a clumsy solution. Why do I want to do this? Well, I've re-oriented my menu so it's vertical. And I have created nice backgrounds on hover with CSS, so the list items are a bit like buttons. But, you still have to click on the actual word in the menu list item (which is what the anchor tag wraps) and you can't click the rest of the block list item that is highlighted on hover (because the a doesn't wrap the li). In short, it's a usability problem when something highlights that you can't click. Does anybody know how I would alter the code (via my own functions.php) that generates wp_nav_menu (it's in includes/nav-menu-template - function start_el - line 67)? to switch these tags around? <pre><code>function start_el(&$output, $item, $depth, $args) { global $wp_query; $indent = ( $depth ) ? str_repeat( "\t", $depth ) : ''; $class_names = $value = ''; $classes = empty( $item->classes ) ? array() : (array) $item->classes; $classes[] = 'menu-item-' . $item->ID; $class_names = join( ' ', apply_filters( 'nav_menu_css_class', array_filter( $classes ), $item, $args ) ); $class_names = ' class="' . esc_attr( $class_names ) . '"'; $id = apply_filters( 'nav_menu_item_id', 'menu-item-'. $item->ID, $item, $args ); $id = strlen( $id ) ? ' id="' . esc_attr( $id ) . '"' : ''; $output .= $indent . '<li' . $id . $value . $class_names .'>'; $attributes = ! empty( $item->attr_title ) ? ' title="' . esc_attr( $item->attr_title ) .'"' : ''; $attributes .= ! empty( $item->target ) ? ' target="' . esc_attr( $item->target ) .'"' : ''; $attributes .= ! empty( $item->xfn ) ? ' rel="' . esc_attr( $item->xfn ) .'"' : ''; $attributes .= ! empty( $item->url ) ? ' href="' . esc_attr( $item->url ) .'"' : ''; $item_output = $args->before; $item_output .= '<a'. $attributes .'>'; $item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after; $item_output .= '</a>'; $item_output .= $args->after; $output .= apply_filters( 'walker_nav_menu_start_el', $item_output, $item, $depth, $args ); } /** * @see Walker::end_el() * @since 3.0.0 * * @param string $output Passed by reference. Used to append additional content. * @param object $item Page data object. Not used. * @param int $depth Depth of page. Not Used. */ function end_el(&$output, $item, $depth) { $output .= "</li>\n"; } }</code></pre> I'm not confident that I can create something that will work... helgatheviking on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-12323 Wed, 12 May 2010 03:17:47 +0000 helgatheviking 12323@http://themeshaper.com/forums/ wish I could edit my post for it's glaring typo (function name mismatch): <pre><code>function remove_access() { if (is_page_template('affiliate.php')) { remove_action('thematic_header','thematic_access',9); } add_action('wp_head','remove_access'); }</code></pre> BenParis on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-12254 Mon, 10 May 2010 02:15:32 +0000 BenParis 12254@http://themeshaper.com/forums/ I'll try these. Thank you helgatheviking on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-12200 Fri, 07 May 2010 04:52:06 +0000 helgatheviking 12200@http://themeshaper.com/forums/ caveat that the first snippet works for me on a page where the slug is a-video-post and the second snippet works on any page that is using my fullpage.php template assuming your affiliate template is affiliate.php you'd need: <pre><code>function remove_menu() { if (is_page_template('affiliate.php')) { remove_action('thematic_header','thematic_access',9); } add_action('wp_head','remove_access'); }</code></pre> helgatheviking on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-12199 Fri, 07 May 2010 04:52:05 +0000 helgatheviking 12199@http://themeshaper.com/forums/ caveat that the first snippet works for me on a page where the slug is a-video-post and the second snippet works on any page that is using my fullpage.php template assuming your affiliate template is affiliate.php you'd need: <pre><code>function remove_menu() { if (is_page_template('affiliate.php')) { remove_action('thematic_header','thematic_access',9); } add_action('wp_head','remove_access'); }</code></pre> helgatheviking on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-12196 Fri, 07 May 2010 01:50:11 +0000 helgatheviking 12196@http://themeshaper.com/forums/ For your template condition you have to use the name of your template file... so template.php and not just template. As for the page condition, I can only guess your page slug is not 'affiliate' <pre><code>function remove_access() { if (is_page('a-video-page')){ remove_action('thematic_header','thematic_access',9); } else { // do nothing } } add_action('template_redirect', 'remove_access');</code></pre> <pre><code>function remove_access() { if (is_page_template('fullpage.php')){ remove_action('thematic_header','thematic_access',9); } else { // do nothing } } add_action('wp_head', 'remove_access');</code></pre> Both of those work for me. BenParis on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-12194 Fri, 07 May 2010 01:20:41 +0000 BenParis 12194@http://themeshaper.com/forums/ thank you in advance thematic geniuses Helga has already offered some help, much appreciated! i'm trying to remove the menu from one page. I've tried adding the following to my funcitions.php file with no luck function remove_access() { if (is_page('affiliate')) { remove_action('thematic_header','thematic_access',9); } add_action('template_redirect', 'remove_access'); } AND THIS function remove_menu() { if (is_page('affiliate')) { remove_action('thematic_header','thematic_access',9); } add_action('template_redirect', 'remove_menu'); } AND THIS created a template called 'affiliate' then added this to functions.php function remove_menu() { if (is_page_template('affiliate')) { remove_action('thematic_header','thematic_access',9); } add_action('wp_head','remove_access'); } Don't know what i'm doint wrong. I can unconditionally remove the menu. helgatheviking on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-11591 Tue, 20 Apr 2010 15:27:01 +0000 helgatheviking 11591@http://themeshaper.com/forums/ Almost posted a new thread b/c I couldn't get this to work w/ if (is_page_template()). But checked out the link Gene posted to the WP action hooks and found that to conditionally remove #access based on a template then you have to edit Val's code to: add_action('wp_head','remove_access'); do so love finding answers in the archives jmslbam on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-10783 Wed, 17 Mar 2010 15:58:33 +0000 jmslbam 10783@http://themeshaper.com/forums/ Hi Val & em hr, It works like a charm! Thanks for posting your solution! <pre> function page_checking() { if(is_front_page()){ add_action('the_content', 'front_page_content'); } } //as suggested by Gene add action to template_redirect add_action('template_redirect', 'page_checking'); function front_page_content() { //you can work your magic here so it only shows the returned string on the front_page return 'x'; } ?> </pre> Val on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-10479 Fri, 05 Mar 2010 15:25:41 +0000 Val 10479@http://themeshaper.com/forums/ @Gene, Thank you! Your suggestion launched an inquiry and solved my problem. I read through parts of the Codex: template hierarchy, API Plugins, actions vs filters and one WordPress forum post from mrfields explaining the difference between actions and filters (understand "objectively"; don't get it intuitively. Still unclear as to how to implement a filter. Shelved for later.) You're correct! init does fire way too soon for checking is_home()! If I understand what I've read, template_loader.php makes a call to the get_home_template function in theme.php. While that function is filterable, I'm not concerned with acting on that hook. (Would that be the correct way to say that?) I'm concerned with overriding the thematic_access function in header-extensions.php when home.php is loaded. Here is the path through the darkness: <pre><code>function remove_access() { // testing implicitly for home.php removes thematic_access from all files. Hmmm. // ? value of $template remains home.php for every subsequent query? // This does not work. Try something else. // if ($template = 'home.php') { // // Testing around is_home() appears to work, leaving thematic_access in place // on all other files. So far. Use this. // if is_home() { { remove_action('thematic_header','thematic_access',9); } } // close the function // // Take action // FYI substituting wp_head for template_redirect "works" // but seems to bog down the loading of the site // Here's what works now // add_action('template_redirect','remove_access');</code></pre> Onward! Val flick on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-10465 Fri, 05 Mar 2010 00:45:31 +0000 flick 10465@http://themeshaper.com/forums/ @Gene: Thanks for the correction :) I should have checked the tag before posting. Luckily it seems I can still edit the post so I've changed it in case someone does a straight copy/paste in the future. /me needs to investigate action hooks more. em hr on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-10463 Fri, 05 Mar 2010 00:36:19 +0000 em hr 10463@http://themeshaper.com/forums/ Hi Val, I've run into this as well. It was hard for me hard to wrap my head around it at first. The problem is that by hooking into <code>init</code> the conditional tags will always return false. Basically, <code>init</code> fires too soon. Try hooking into <code>template_redirect</code> and your function will work the way you've intended it. Look to the codex for more info on action hooks: <a href="http://codex.wordpress.org/Plugin_API/Action_Reference" rel="nofollow">http://codex.wordpress.org/Plugin_API/Action_Reference</a> @ Flick - Good call :) ...but the proper conditional tag for both "blog" and "front page" would be typed <code>is_front_page</code> that missing underscore will throw an error for calling an undefined function. Cheers, Gene flick on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-10462 Thu, 04 Mar 2010 23:50:37 +0000 flick 10462@http://themeshaper.com/forums/ Perhaps you can consider using <code>is_front_page</code> instead? From what I've understood, if a static page is used, it's best to use <code>is_front_page</code> because the <code>is_home</code> looks for the 'blog' (or default index) page. Chris also provided an interesting piece of code <a href="http://themeshaper.com/forums/topic/is-this-right-way-of-adding-page-template-to-display-posts-from-a-category#post-10224">here</a> using <pre><code>if (is_page_template('blog.php'))</code></pre> I haven't tested it myself, but perhaps it might be useful for you? ----------------------------------------------- EDIT following Gene's post: corrected is_front_page Val on "Conditionally Removing thematic_access" http://themeshaper.com/forums/topic/conditionally-removing-thematic_access#post-10459 Thu, 04 Mar 2010 23:38:14 +0000 Val 10459@http://themeshaper.com/forums/ I created home.php and uploaded to my child theme directory. I can conditionally add stylesheets to the head by checking if is_home() is true. I can conditionally add a "welcome" message in the header section. But I cannot conditionally remove thematic_access. It's all or nothing: I can remove it unconditionally, only. Any ideas as to why this code does not work? [code] // remove thematic_access() function remove_access() { if (is_home()) { remove_action('thematic_header','thematic_access',9); } } add_action('init', 'remove_access'); [code] Thank you!