WordPress.org

Make WordPress Core

Ticket #23770: 23770.6.diff

File 23770.6.diff, 12.2 KB (added by DrewAPicture, 5 years ago)

Match menu ids to Edit link + RTL

  • wp-admin/css/wp-admin-rtl.css

     
    697697.fixed .column-comments a {
    698698        float: right;
    699699}
     700.fixed .column-menus {
     701        text-align: right;
     702}
    700703.sorting-indicator {
    701704        margin-left: 0;
    702705        margin-right: 7px;
     
    20832086        float: right;
    20842087}
    20852088
     2089.menu-location-menus select {
     2090        float: right;
     2091}
     2092
     2093.locations-row-links {
     2094        float: right;
     2095        margin: 4px 6px 0 0;
     2096}
     2097
     2098.locations-add-menu-link {
     2099        direction: rtl;
     2100}
     2101
     2102.locations-edit-menu-link {
     2103        border-left: 1px solid #CCCCCC;
     2104        border-right: 0;
     2105        padding-left: 6px;
     2106        padding-right: 0;
     2107        float: right;
     2108}
     2109
    20862110/* Menu Container */
    20872111#menu-management-liquid {
    20882112        float: right;
  • wp-admin/css/wp-admin.css

     
    24232423        width: 8%;
    24242424}
    24252425
     2426.fixed .column-posts,
    24262427.fixed .column-date,
    24272428.fixed .column-parent,
    24282429.fixed .column-links,
     
    24392440        width: 15%;
    24402441}
    24412442
     2443.fixed .column-slug {
     2444        width: 25%;
     2445}
     2446
     2447.fixed .column-locations {
     2448        width: 35%;
     2449}
     2450
    24422451.fixed .column-comments {
    24432452        width: 4em;
    24442453        padding: 8px 0;
     
    24532462        float: left;
    24542463}
    24552464
    2456 .fixed .column-slug {
    2457         width: 25%;
    2458 }
    2459 
    2460 .fixed .column-posts {
    2461         width: 10%;
    2462 }
    2463 
    24642465.fixed .column-icon {
    24652466        width: 80px;
    24662467}
     
    25472548        margin-top: 3px;
    25482549}
    25492550
     2551#menu-locations-wrap .widefat {
     2552        width: 60%;
     2553}
     2554
    25502555.widefat th.sortable,
    25512556.widefat th.sorted {
    25522557        padding: 0;
     
    73807385
    73817386#nav-menus-frame {
    73827387        margin-left: 300px;
    7383         margin-top: 28px;
     7388        margin-top: 23px;
    73847389}
    73857390
    73867391#wpbody-content #menu-settings-column {
     
    73977402        margin: 10px 0 0;
    73987403}
    73997404
    7400 .metabox-holder-disabled .postbox {
     7405.metabox-holder-disabled .postbox,
     7406.metabox-holder-disabled .accordion-section-content  {
    74017407        opacity: 0.5;
    74027408        filter: alpha(opacity=50);
    74037409}
     
    74287434}
    74297435
    74307436.manage-menus {
    7431         margin-top: 13px;
     7437        margin-top: 23px;
    74327438        padding: 10px;
    74337439        overflow: hidden;
    74347440        -webkit-border-radius: 3px;
     
    75627568        margin-top: -2px;
    75637569}
    75647570
     7571.widefat td.menu-location-menus {
     7572        padding-bottom: 5px;
     7573}
     7574
     7575.menu-location-menus select {
     7576        float: left;
     7577}
     7578
     7579#locations-nav-menu-wrapper {
     7580        padding: 5px 0;
     7581}
     7582
     7583.locations-nav-menu-select select {
     7584        float: left;
     7585        width: 160px;
     7586        margin-right: 5px;
     7587}
     7588
     7589.locations-row-links {
     7590        float: left;
     7591        margin: 6px 0 0 6px;
     7592}
     7593
     7594.locations-edit-menu-link,
     7595.locations-add-menu-link {
     7596        margin: 0 3px;
     7597}
     7598
     7599.locations-edit-menu-link {
     7600        padding-right: 3px;
     7601        border-right: 1px solid #ccc;
     7602}
     7603
    75657604#wpbody .open-label {
    75667605        display: block;
    75677606        float:left;
  • wp-admin/js/nav-menu.js

     
    4848                        if ( menus.oneThemeLocationNoMenus )
    4949                                $( '#posttype-page' ).addSelectedToMenu( api.addMenuItemToBottom );
    5050
     51                        this.initManageLocations();
     52
    5153                        this.initAccessibility();
    5254
    5355                        this.initToggles();
     
    735737                        }
    736738                },
    737739
     740                initManageLocations : function () {
     741                        $('.menu-location-menus select').on('change', function () {
     742                                var editLink = $('.locations-edit-menu-link');
     743                                if ($(this).find('option:selected').data('orig'))
     744                                        editLink.show();
     745                                else
     746                                        editLink.hide();
     747                        });
     748                },
     749
    738750                attachMenuEditListeners : function() {
    739751                        var that = this;
    740752                        $('#update-nav-menu').bind('click', function(e) {
     
    927939                },
    928940
    929941                attachUnsavedChangesListener : function() {
    930                         $('#menu-management input, #menu-management select, #menu-management, #menu-management textarea').change(function(){
     942                        $('#menu-management input, #menu-management select, #menu-management, #menu-management textarea, .menu-location-menus select').change(function(){
    931943                                api.registerChange();
    932944                        });
    933945
    934                         if ( 0 != $('#menu-to-edit').length ) {
     946                        if ( 0 != $('#menu-to-edit').length || 0 != $('.menu-location-menus select').length ) {
    935947                                window.onbeforeunload = function(){
    936948                                        if ( api.menusChanged )
    937949                                                return navMenuL10n.saveAlert;
  • wp-admin/nav-menus.php

     
    3737// The menu id of the current menu being edited
    3838$nav_menu_selected_id = isset( $_REQUEST['menu'] ) ? (int) $_REQUEST['menu'] : 0;
    3939
     40// Get existing menu locations assignments
     41$locations = get_registered_nav_menus();
     42$menu_locations = get_nav_menu_locations();
     43$num_locations = count( array_keys( $locations ) );
     44
    4045// Allowed actions: add, update, delete
    4146$action = isset( $_REQUEST['action'] ) ? $_REQUEST['action'] : 'edit';
    4247
     
    262267        case 'update':
    263268                check_admin_referer( 'update-nav_menu', 'update-nav-menu-nonce' );
    264269
    265                 // Get existing menu locations assignments
    266                 $locations = get_registered_nav_menus();
    267                 $menu_locations = get_nav_menu_locations();
    268 
    269270                // Remove menu locations that have been unchecked
    270271                foreach ( $locations as $location => $description ) {
    271272                        if ( ( empty( $_POST['menu-locations'] ) || empty( $_POST['menu-locations'][ $location ] ) ) && isset( $menu_locations[ $location ] ) && $menu_locations[ $location ] == $nav_menu_selected_id )
     
    341342                        }
    342343                }
    343344                break;
     345        case 'locations':
     346                add_filter( 'screen_options_show_screen', '__return_false' );
     347
     348                if ( isset( $_POST['menu-locations'] ) ) {
     349                        check_admin_referer( 'save-menu-locations' );
     350
     351                        $new_menu_locations = array_map( 'absint', $_POST['menu-locations'] );
     352                        $menu_locations = array_merge( $menu_locations, $new_menu_locations );
     353                        // Set menu locations
     354                        set_theme_mod( 'nav_menu_locations', $menu_locations );
     355
     356                        $messages[] = '<div id="message" class="updated"><p>' . __( 'Menu locations updated.' ) . '</p></div>';
     357                }
     358                break;
    344359}
    345360
    346361// Get all nav menus
     
    350365// Are we on the add new screen?
    351366$add_new_screen = ( isset( $_GET['menu'] ) && 0 == $_GET['menu'] ) ? true : false;
    352367
     368$locations_screen = ( isset( $_GET['action'] ) && 'locations' == $_GET['action'] ) ? true : false;
     369
    353370// If we have one theme location, and zero menus, we take them right into editing their first menu
    354371$page_count = wp_count_posts( 'page' );
    355372$one_theme_location_no_menus = ( 1 == count( get_registered_nav_menus() ) && ! $add_new_screen && empty( $nav_menus ) && ! empty( $page_count->publish ) ) ? true : false;
     
    474491?>
    475492<div class="wrap">
    476493        <?php screen_icon(); ?>
    477         <h2><?php _e( 'Menus' ); ?> <a href="<?php echo esc_url( add_query_arg( array( 'action' => 'edit', 'menu' => 0, ), admin_url( 'nav-menus.php' ) ) ); ?>" class="add-new-h2"><?php _ex( 'Add New', 'menu' ); ?></a></h2>
     494        <h2 class="nav-tab-wrapper">
     495                <a href="<?php echo admin_url( 'nav-menus.php' ); ?>" class="nav-tab<?php if ( ! isset( $_GET['action'] ) || isset( $_GET['action'] ) && 'locations' != $_GET['action'] ) echo ' nav-tab-active'; ?>"><?php esc_html_e( 'Edit Menus' ); ?></a>
     496                <a href="<?php echo esc_url( add_query_arg( array( 'action' => 'locations' ), admin_url( 'nav-menus.php' ) ) ); ?>" class="nav-tab<?php if ( $locations_screen ) echo ' nav-tab-active'; ?>"><?php esc_html_e( 'Manage Locations' ); ?></a>
     497        </h2>
    478498        <?php
    479499        foreach( $messages as $message ) :
    480500                echo $message . "\n";
    481501        endforeach;
    482502        ?>
     503        <?php
     504        if ( $locations_screen ) :
     505                echo '<p>' . sprintf( _n( 'Your theme supports %s menu. Select which menu you would like to use below.', 'Your theme supports %s menus. Select a menu to use for each theme location below.', $num_locations ), number_format_i18n( $num_locations ) ) . '</p>';
     506        ?>
     507        <div id="menu-locations-wrap">
     508                <form method="post" action="<?php echo esc_url( add_query_arg( array( 'action' => 'locations' ), admin_url( 'nav-menus.php' ) ) ); ?>">
     509                        <p class="button-controls"><?php submit_button( __( 'Save Changes' ), 'primary left', 'nav-menu-locations', false ); ?></p>
     510                        <table class="widefat fixed" cellspacing="0" id="menu-locations-table">
     511                                <thead>
     512                                <tr>
     513                                        <th scope="col" class="manage-column column-locations"><?php _e( 'Theme Location' ); ?></th>
     514                                        <th scope="col" class="manage-column column-menus"><?php _e( 'Assigned Menu' ); ?></th>
     515                                </tr>
     516                                </thead>
     517                                <tfoot>
     518                                <tr>
     519                                        <th scope="col" class="manage-column column-locations"><?php _e( 'Theme Location' ); ?></th>
     520                                        <th scope="col" class="manage-column column-menus"><?php _e( 'Assigned Menu' ); ?></th>
     521                                </tr>
     522                                </tfoot>
     523                                <tbody class="menu-locations">
     524                                <?php foreach ( $locations as $_location => $_name ) { ?>
     525                                        <tr id="menu-locations-row">
     526                                                <td class="menu-location-title"><strong><?php echo $_name; ?></strong></td>
     527                                                <td class="menu-location-menus">
     528                                                        <select name="menu-locations[<?php echo $_location; ?>]" id="locations-<?php echo $_location; ?>">
     529                                                                <option value="0"><?php printf( '&mdash; %s &mdash;', esc_html__( 'Select a Menu' ) ); ?></option>
     530                                                                <?php foreach ( $nav_menus as $menu ) : ?>
     531                                                                        <?php $selected = isset( $menu_locations[$_location] ) && $menu_locations[$_location] == $menu->term_id; ?>
     532                                                                        <option <?php if ( $selected ) echo 'data-orig="true"'; ?> <?php selected( $selected ); ?> value="<?php echo $menu->term_id; ?>">
     533                                                                                <?php $truncated_name = wp_html_excerpt( $menu->name, 40 );
     534                                                                                echo $truncated_name == $menu->name ? $menu->name : trim( $truncated_name ) . '&hellip;'; ?>
     535                                                                        </option>
     536                                                                <?php endforeach; ?>
     537                                                        </select>
     538                                                        <div class="locations-row-links">
     539                                                                <?php if ( isset( $menu_locations[ $_location ] ) && 0 != $menu_locations[ $_location ] ) : ?>
     540                                                                <span class="locations-edit-menu-link">
     541                                                                        <a href="<?php echo esc_url( add_query_arg( array( 'action' => 'edit', 'menu' => $menu_locations[$_location] ), admin_url( 'nav-menus.php' ) ) ); ?>">
     542                                                                                <?php _ex( 'Edit', 'menu' ); ?>
     543                                                                        </a>
     544                                                                </span>
     545                                                                <?php endif; ?>
     546                                                                <span class="locations-add-menu-link">
     547                                                                        <a href="<?php echo esc_url( add_query_arg( array( 'action' => 'edit', 'menu' => 0 ), admin_url( 'nav-menus.php' ) ) ); ?>">
     548                                                                                <?php _ex( 'Use new menu', 'menu' ); ?>
     549                                                                        </a>
     550                                                                </span>
     551                                                        </div><!-- #locations-row-links -->
     552                                                </td><!-- .menu-location-menus -->
     553                                        </tr><!-- #menu-locations-row -->
     554                                <?php } // foreach ?>
     555                                </tbody>
     556                        </table>
     557                        <p class="button-controls"><?php submit_button( __( 'Save Changes' ), 'primary left', 'nav-menu-locations', false ); ?></p>
     558                        <?php wp_nonce_field( 'save-menu-locations' ); ?>
     559                        <input type="hidden" name="menu" id="nav-menu-meta-object-id" value="<?php echo esc_attr( $nav_menu_selected_id ); ?>" />
     560                </form>
     561        </div><!-- #menu-locations-wrap -->
     562        <?php do_action( 'after_menu_locations_table' ); ?>
     563        <?php else : ?>
    483564        <div class="manage-menus">
    484         <?php if ( $menu_count < 2 ) : ?>
     565                <?php if ( $menu_count < 2 ) : ?>
    485566                <span class="add-edit-menu-action">
    486567                        <?php printf( __( 'Edit your menu below, or <a href="%s">create a new menu</a>.' ), esc_url( add_query_arg( array( 'action' => 'edit', 'menu' => 0 ), admin_url( 'nav-menus.php' ) ) ) ); ?>
    487568                </span><!-- /add-edit-menu-action -->
    488         <?php else : ?>
    489                 <form method="post" action="<?php echo admin_url( 'nav-menus.php' ); ?>">
     569                <?php else : ?>
     570                <form method="get" action="<?php echo admin_url( 'nav-menus.php' ); ?>">
    490571                        <input type="hidden" name="action" value="edit" />
    491572                        <label for="menu" class="selected-menu"><?php _e( 'Select a menu to edit:' ); ?></label>
    492573                        <select name="menu" id="menu">
     
    520601                                <?php printf( __( 'or <a href="%s">create a new menu</a>.' ), esc_url( add_query_arg( array( 'action' => 'edit', 'menu' => 0 ), admin_url( 'nav-menus.php' ) ) ) ); ?>
    521602                        </span><!-- /add-new-menu-action -->
    522603                </form>
    523         <?php endif; ?>
     604                <?php endif; ?>
    524605        </div><!-- /manage-menus -->
    525606        <div id="nav-menus-frame">
    526607        <div id="menu-settings-column" class="metabox-holder<?php if ( isset( $_GET['menu'] ) && '0' == $_GET['menu'] ) { echo ' metabox-holder-disabled'; } ?>">
     
    633714                </div><!-- /#menu-management -->
    634715        </div><!-- /#menu-management-liquid -->
    635716        </div><!-- /#nav-menus-frame -->
     717        <?php endif; ?>
    636718</div><!-- /.wrap-->
    637719<?php include( './admin-footer.php' ); ?>