help setting accesskeys

[minute44]

I want to set accesskeys for my main navigation. I can do it if I substitute the PHP it has that builds the list of pages with static HTML but if I do that I lose the current page link state.

At the moment the Nav is like this:

PHP: Select all

<ul>
<?php if (is_home()) { ?>
<li class="current_page_item"><a href="<?php bloginfo('home'); ?>">Home</a></li>
<?php } else { ?>
<li><a href="<?php bloginfo('home'); ?>">Home</a></li>
<?php } ?>
<?php wp_list_pages('sort_column=menu_order&depth=1&title_li='); ?>
</ul>

I can change it to this:

HTML: Select all

<ul>
<li><a href="http://www.minute44.com" accesskey="1">Home</a></li>
<li><a href="http://www.minute44.com/about" accesskey="2">About Me</a></li>
<li><a href="http://www.minute44.com/galleries" accesskey="3">Galleries</a></li>
<li><a href="http://www.minute44.com/shout-up" accesskey="4">Shout Up</a></li>
</ul>

but obviously I lose the effect of the current page link appearing different.

Can anyone see a way for me to alter the php in the first one so that it assigns the links accesskeys 1-4?

Cheers

Dan

[c010depunkk]

Well the first one is easy:

HTML: Select all

<ul>
<?php if (is_home()) { ?>
<li class="current_page_item"><a href="<?php bloginfo('home'); ?>" accesskey="1">Home</a></li>
<?php } else { ?>
<li><a href="<?php bloginfo('home'); ?>" accesskey="1">Home</a></li>
<?php } ?>
<?php wp_list_pages('sort_column=menu_order&depth=1&title_li='); ?>
</ul>

For the rest of the links, you'll have to modify the "wp_list_pages" function. If you post the code here I'm sure we can figure something out....

[minute44]

Just trying to find the file that contains the function.

[minute44]

OK, here's the function:

PHP: Select all

function wp_list_pages($args = '') {
    $defaults = array(
        'depth' => 0, 'show_date' => '',
        'date_format' => get_option('date_format'),
        'child_of' => 0, 'exclude' => '',
        'title_li' => __('Pages'), 'echo' => 1,
        'authors' => '', 'sort_column' => 'menu_order, post_title'
    );

    $r = wp_parse_args( $args, $defaults );
    extract( $r, EXTR_SKIP );

    $output = '';
    $current_page = 0;

    // sanitize, mostly to keep spaces out
    $r['exclude'] = preg_replace('[^0-9,]', '', $r['exclude']);

    // Allow plugins to filter an array of excluded pages
    $r['exclude'] = implode(',', apply_filters('wp_list_pages_excludes', explode(',', $r['exclude'])));

    // Query pages.
    $pages = get_pages($r);

    if ( !empty($pages) ) {
        if ( $r['title_li'] )
            $output .= '<li class="pagenav">' . $r['title_li'] . '<ul>';

        global $wp_query;
        if ( is_page() )
            $current_page = $wp_query->get_queried_object_id();
        $output .= walk_page_tree($pages, $r['depth'], $current_page, $r);

        if ( $r['title_li'] )
            $output .= '</ul></li>';
    }

    $output = apply_filters('wp_list_pages', $output);

    if ( $r['echo'] )
        echo $output;
    else
        return $output;
} 

[c010depunkk]

OK, As far as i can tell the pages get added here:

PHP: Select all

$output .= walk_page_tree($pages, $r['depth'], $current_page, $r); 

Let's check out that function.....

[minute44]

You know what? I can't be arsed with this anymore. Too much work to add a farting little feature that is useless to 99.99% of users and could very well harm my existing accessibility.

[c010depunkk]

ok....