WordPress.org

Make WordPress Core

Ticket #23770: 23770.6.diff

File 23770.6.diff, 12.2 KB (added by DrewAPicture, 2 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' ); ?>