WordPress.org
Get WordPress

Make WordPress Core

Changeset 14482


Ignore:
Timestamp:
05/06/2010 07:19:53 PM (14 years ago)
Author:
nacin
Message:

Ensure valid HTML in the nav menu UI. props ocean90, see #13220.

Location:
trunk/wp-admin
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/includes/nav-menu.php

    r14471 r14482  
    155155                    <label for="edit-menu-item-description-<?php echo $item_id; ?>">
    156156                        <?php _e( 'Description' ); ?><br />
    157                         <textarea id="edit-menu-item-description-<?php echo $item_id; ?>" class="widefat edit-menu-item-description" rows="3" name="menu-item-description[<?php echo $item_id; ?>]"><?php echo esc_html( $item->description ); ?></textarea>
     157                        <textarea id="edit-menu-item-description-<?php echo $item_id; ?>" class="widefat edit-menu-item-description" rows="3" cols="20" name="menu-item-description[<?php echo $item_id; ?>]"><?php echo esc_html( $item->description ); ?></textarea>
    158158                        <span class="description"><?php _e('The description will be displayed in the menu if the current theme supports it.'); ?></span>
    159159                    </label>
    160160                </p>
    161161
    162                 <span class="menu-item-actions description-wide submitbox">
     162                <div class="menu-item-actions description-wide submitbox">
    163163                    <?php if( 'custom' != $item->type ) : ?>
    164164                        <p class="link-to-original"><?php
    … …  
    183183                    ); ?>"><?php _e('Remove'); ?></a> | <a class="item-close submitclose" id="close-<?php echo $item_id; ?>" href="<?php echo admin_url( 'nav-menus.php' ); ?>"><?php _e('Close'); ?></a>
    184184                    <input class="button-primary save-menu-item" name="save_menu_item" type="submit" value="<?php esc_attr_e('Save Menu Item'); ?>" />
    185                 </span>
     185                </div>
    186186
    187187                <input class="menu-item-data-append" type="hidden" name="menu-item-append[<?php echo $item_id; ?>]" value="<?php echo $item->append; ?>" />
    … …  
    190190                <input class="menu-item-data-object" type="hidden" name="menu-item-object[<?php echo $item_id; ?>]" value="<?php echo esc_attr( $item->object ); ?>" />
    191191                <input class="menu-item-data-parent-id" type="hidden" name="menu-item-parent-id[<?php echo $item_id; ?>]" value="<?php echo esc_attr( $item->menu_item_parent ); ?>" />
    192                 <input class="menu-item-data-position" type="hidden" class="menu-item-position" name="menu-item-position[<?php echo $item_id; ?>]" value="<?php echo esc_attr( $item->menu_order ); ?>" />
     192                <input class="menu-item-data-position" type="hidden" name="menu-item-position[<?php echo $item_id; ?>]" value="<?php echo esc_attr( $item->menu_order ); ?>" />
    193193                <input class="menu-item-data-type" type="hidden" name="menu-item-type[<?php echo $item_id; ?>]" value="<?php echo esc_attr( $item->type ); ?>" />
    194194            </div><!-- .menu-item-settings-->
    … …  
    567567    <div id="posttype-<?php echo $post_type_name; ?>" class="posttypediv">
    568568        <ul id="posttype-<?php echo $post_type_name; ?>-tabs" class="posttype-tabs add-menu-item-tabs">
    569             <li <?php echo ( 'all' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo add_query_arg($post_type_name . '-tab', 'all', remove_query_arg($removed_args)); ?>#<?php echo $post_type_name; ?>-all"><?php _e('View All'); ?></a></li>
    570             <li <?php echo ( 'search' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo add_query_arg($post_type_name . '-tab', 'search', remove_query_arg($removed_args)); ?>#tabs-panel-posttype-<?php echo $post_type_name; ?>-search"><?php _e('Search'); ?></a></li>
     569            <li <?php echo ( 'all' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo esc_url(add_query_arg($post_type_name . '-tab', 'all', remove_query_arg($removed_args))); ?>#<?php echo $post_type_name; ?>-all"><?php _e('View All'); ?></a></li>
     570            <li <?php echo ( 'search' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo esc_url(add_query_arg($post_type_name . '-tab', 'search', remove_query_arg($removed_args))); ?>#tabs-panel-posttype-<?php echo $post_type_name; ?>-search"><?php _e('Search'); ?></a></li>
    571571        </ul>
    572572
    … …  
    630630            <span class="list-controls">
    631631                <a href="<?php
    632                     echo add_query_arg(
     632                    echo esc_url(add_query_arg(
    633633                        array(
    634634                            $post_type_name . '-tab' => 'all',
    … …  
    636636                        ),
    637637                        remove_query_arg($removed_args)
    638                     );
     638                    ));
    639639                ?>#posttype-<?php echo $post_type_name; ?>" class="select-all"><?php _e('Select All'); ?></a>
    640640            </span>
    … …  
    728728    <div id="taxonomy-<?php echo $taxonomy_name; ?>" class="taxonomydiv">
    729729        <ul id="taxonomy-<?php echo $taxonomy_name; ?>-tabs" class="taxonomy-tabs add-menu-item-tabs">
    730             <li <?php echo ( 'most-used' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo add_query_arg($taxonomy_name . '-tab', 'most-used', remove_query_arg($removed_args)); ?>#tabs-panel-<?php echo $taxonomy_name; ?>-pop"><?php _e('Most Used'); ?></a></li>
    731             <li <?php echo ( 'search' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo add_query_arg($taxonomy_name . '-tab', 'search', remove_query_arg($removed_args)); ?>#tabs-panel-search-taxonomy-<?php echo $taxonomy_name; ?>"><?php _e('Search'); ?></a></li>
    732             <li <?php echo ( 'all' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo add_query_arg($taxonomy_name . '-tab', 'all', remove_query_arg($removed_args)); ?>#tabs-panel-<?php echo $taxonomy_name; ?>-all"><?php _e('View All'); ?></a></li>
     730            <li <?php echo ( 'most-used' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo esc_url(add_query_arg($taxonomy_name . '-tab', 'most-used', remove_query_arg($removed_args))); ?>#tabs-panel-<?php echo $taxonomy_name; ?>-pop"><?php _e('Most Used'); ?></a></li>
     731            <li <?php echo ( 'search' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo esc_url(add_query_arg($taxonomy_name . '-tab', 'search', remove_query_arg($removed_args))); ?>#tabs-panel-search-taxonomy-<?php echo $taxonomy_name; ?>"><?php _e('Search'); ?></a></li>
     732            <li <?php echo ( 'all' == $current_tab ? ' class="tabs"' : '' ); ?>><a class="nav-tab-link" href="<?php echo esc_url(add_query_arg($taxonomy_name . '-tab', 'all', remove_query_arg($removed_args))); ?>#tabs-panel-<?php echo $taxonomy_name; ?>-all"><?php _e('View All'); ?></a></li>
    733733        </ul>
    734734
    … …  
    798798            <span class="list-controls">
    799799                <a href="<?php
    800                     echo add_query_arg(
     800                    echo esc_url(add_query_arg(
    801801                        array(
    802802                            $taxonomy_name . '-tab' => 'all',
    … …  
    804804                        ),
    805805                        remove_query_arg($removed_args)
    806                     );
     806                    ));
    807807                ?>#taxonomy-<?php echo $taxonomy_name; ?>" class="select-all"><?php _e('Select All'); ?></a>
    808808            </span>

  • trunk/wp-admin/nav-menus.php

    r14450 r14482  
    422422        <div id="menu-management">
    423423            <div id="select-nav-menu-container" class="hide-if-js">
    424                 <form id="select-nav-menu">
     424                <form id="select-nav-menu" action="">
    425425                    <strong><label for="select-nav-menu"><?php esc_html_e( 'Select Menu: ' ); ?></label></strong>
    426                     <select id="select-nav-menu" name="menu">
     426                    <select class="select-nav-menu" name="menu">
    427427                        <?php foreach( (array) $nav_menus as $_nav_menu ) : ?>
    428428                            <option value="<?php echo esc_attr($_nav_menu->term_id) ?>" <?php selected($nav_menu_selected_id, $_nav_menu->term_id); ?>>
    … …  
    442442
    443443                    ?><a href="<?php
    444                         echo add_query_arg(
     444                        echo esc_url(add_query_arg(
    445445                            array(
    446446                                'action' => 'edit',
    … …  
    448448                            ),
    449449                            admin_url( 'nav-menus.php' )
    450                         );
     450                        ));
    451451                    ?>" class="nav-tab<?php
    452452                        if ( $nav_menu_selected_id == $_nav_menu->term_id )
    … …  
    457457                endforeach;
    458458                ?><a href="<?php
    459                     echo add_query_arg(
     459                    echo esc_url(add_query_arg(
    460460                        array(
    461461                            'action' => 'edit',
    … …  
    463463                        ),
    464464                        admin_url( 'nav-menus.php' )
    465                     );
     465                    ));
    466466                ?>" class="nav-tab menu-add-new<?php
    467467                    if ( 0 == $nav_menu_selected_id )
    … …  
    477477                                <label class="menu-name-label howto open-label" for="menu-name">
    478478                                    <span><?php _e('Menu Name'); ?></span>
    479                                     <input name="menu-name" type="text" class="menu-name regular-text menu-item-textbox input-with-default-title" title="Enter menu name here." value="<?php echo esc_attr( $nav_menu_selected_title  ); ?>" />
     479                                    <input name="menu-name" id="menu-name" type="text" class="menu-name regular-text menu-item-textbox input-with-default-title" title="Enter menu name here." value="<?php echo esc_attr( $nav_menu_selected_title  ); ?>" />
    480480                                    <br class="clear" />
    481481                                </label>
    … …  
    487487                                <?php if ( ! empty( $nav_menu_selected_id ) ) : ?>
    488488                                <div class="delete-action">
    489                                     <a class="submitdelete deletion menu-delete" href="<?php echo wp_nonce_url( admin_url('nav-menus.php?action=delete&amp;menu=' . $nav_menu_selected_id), 'delete-nav_menu-' . $nav_menu_selected_id ); ?>"><?php _e('Delete Menu'); ?></a>
     489                                    <a class="submitdelete deletion menu-delete" href="<?php echo esc_url( wp_nonce_url( admin_url('nav-menus.php?action=delete&amp;menu=' . $nav_menu_selected_id), 'delete-nav_menu-' . $nav_menu_selected_id ) ); ?>"><?php _e('Delete Menu'); ?></a>
    490490                                </div><!--END .delete-action-->
    491491                                <?php endif; ?>
Note: See TracChangeset for help on using the changeset viewer.