Changeset 23441

Timestamp:

02/16/2013 04:53:59 AM (11 years ago)

Author:

markjaquith

Message:

Improve the UX of the Nav Menus screen. Kill the tabs, and change to a
dropdown, unless you have zero or one menus (which is the most common),
in which case you jump right into editing your sole menu.

Do assignment to location using checkboxes in the main menu editing
section instead of the backwards menu => location assignment in a
random meta box.

More to come, but this gets us started.

props lessbloat, DrewAPicture, jkudish. see #23119

Location:

trunk/wp-admin

Files:

• css/colors-fresh.css (modified) (2 diffs)
• css/wp-admin-rtl.css (modified) (3 diffs)
• css/wp-admin.css (modified) (9 diffs)
• includes/nav-menu.php (modified) (13 diffs)
• js/nav-menu.js (modified) (14 diffs)
• nav-menus.php (modified) (12 diffs)

trunk/wp-admin/css/colors-fresh.css

@@ -401 +401 @@
 .submitbox .submitdelete:hover,
 #media-items a.delete:hover,
-#media-items a.delete-permanently:hover {
+#media-items a.delete-permanently:hover,
+#nav-menu-footer .menu-delete:hover {
     color: #fff;
     background-color: #f00;
@@ -1682 +1683 @@
     background: #21759b;
     color: #fff;
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }
 /* end added from nav-menu.css */

trunk/wp-admin/css/wp-admin-rtl.css

@@ -45 +45 @@
 26.0 - Full Overlay w/ Sidebar
 27.0 - Customize Loader
+
+
 
 
@@ -2304 +2306 @@
 }
 
-.auto-add-pages {
-    float: right;
-}
-
 /* Star ratings */
 div.star-holder {
@@ -2544 +2542 @@
 }
 
-/**
- * HiDPI Displays
- */
+/*------------------------------------------------------------------------------
+  28.0 - Nav Menus
+------------------------------------------------------------------------------*/
+.nav-menus-php .major-publishing-actions .publishing-action {
+    float: left;
+}
+
+.menu-settings dd {
+    float: right;
+    padding-right: 170px;
+}
+
+.manage-menus span {
+    float: right;
+}
+
+.menu-settings dt {
+    left: auto;
+    right: 0;
+}
+
+.menu-settings dd input,
+.menu-settings dd span,
+.menu-settings dd label,
+.manage-menus select {
+    float: right;
+    margin-right: 6px;
+}
+
+/*------------------------------------------------------------------------------
+  29.0 - HiDPI
+------------------------------------------------------------------------------*/
 @media print,
   (-o-min-device-pixel-ratio: 5/4),

trunk/wp-admin/css/wp-admin.css

@@ -6705 +6705 @@
 /* nav-menu */
 
+
+
+
+
+
+
+
+
 #nav-menus-frame {
     margin-left: 300px;
+
 }
 
@@ -6715 +6724 @@
     clear: both;
     float: left;
-    padding-top: 24px;
-}
-
-.no-js #wpbody-content #menu-settings-column {
-    padding-top: 31px;
+    padding-top: 0;
 }
 
@@ -6740 +6745 @@
 }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 /* Menu Container */
 #menu-management-liquid {
     float: left;
     min-width: 100%;
+
 }
 
@@ -6758 +6842 @@
 
 .nav-menus-php #post-body {
-    padding: 10px;
+    padding: 10px;
     border-width: 1px 0;
     border-style: solid;
@@ -6770 +6854 @@
 #nav-menu-header {
     border-bottom: 1px solid;
+
+
+
+
+
 }
 
@@ -6788 +6877 @@
 #menu-management .menu-add-new abbr {
     font-weight:bold;
-}
-
-/* Menu Tabs */
-
-#menu-management .nav-tabs-nav {
-    margin: 0 20px;
-}
-
-#menu-management .nav-tabs-arrow {
-    width: 10px;
-    padding: 0 5px 4px;
-    cursor: pointer;
-    position: absolute;
-    top: 0;
-    line-height: 22px;
-    font-size: 18px;
-}
-
-#menu-management .nav-tabs-arrow-left {
-    left: 0;
-}
-
-#menu-management .nav-tabs-arrow-right {
-    right: 0;
-    text-align: right;
-}
-
-#menu-management .nav-tabs-wrapper {
-    width: 100%;
-    height: 28px;
-    margin-bottom: -1px;
-    overflow: hidden;
-}
-
-#menu-management .nav-tabs {
-    padding-left: 20px;
-    padding-right: 10px;
-}
-
-.js #menu-management .nav-tabs {
-    float: left;
-    margin-left: 0px;
-    margin-right: -400px;
-}
-
-#menu-management .nav-tab {
-    margin-bottom: 0;
-    font-size: 14px;
 }
 
@@ -7034 +7075 @@
 
 #menu-to-edit {
-    padding: 1em 0;
+    margin: 0;
+    padding: 0.1em 0;
 }
 
@@ -7050 +7092 @@
     line-height:1.5em;
     position:relative;
-    margin: 13px 0 0 0;
+    margin:  0 0;
 }
 
@@ -7270 +7312 @@
     float: right;
     line-height: 23px;
-    margin: 5px 0 1px;
-}
-
-.nav-menus-php .major-publishing-actions .delete-action {
-    vertical-align: middle;
-    text-align: left;
-    float: left;
-    padding-right: 15px;
-    margin-top: 5px;
-}
-
-.menu-name-label span,
-.auto-add-pages label {
-    font-size: 12px;
-    font-style: normal;
-}
-
-.menu-name-label {
-    margin-right: 15px;
-}
-
-.auto-add-pages input {
-    margin-top: 0;
-}
-
-.auto-add-pages {
-    margin-top: 4px;
-    float: left;
+    margin: 2px 0 1px;
+}
+
+.nav-menus-php .blank-slate .menu-settings {
+    display: none;
+}
+
+.nav-menus-php .delete-action {
+    float: left;
+    margin-top: 2px;
 }
 

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

@@ -82 +82 @@
         $title = empty( $item->label ) ? $title : $item->label;
 
+
+
+
+
         ?>
         <li id="menu-item-<?php echo $item_id; ?>" class="<?php echo implode(' ', $classes ); ?>">
             <dl class="menu-item-bar">
                 <dt class="menu-item-handle">
-                    <span class="item-title"><?php echo esc_html( $title ); ?></span>
+                    <span class="item-title"><?php echo esc_html( $title ); ?></span>
                     <span class="item-controls">
                         <span class="item-type"><?php echo esc_html( $item->type_label ); ?></span>
@@ -183 +187 @@
                                 'menu-item' => $item_id,
                             ),
-                            remove_query_arg($removed_args, admin_url( 'nav-menus.php' ) )
+                             )
                         ),
                         'delete-menu_item_' . $item_id
-                    ); ?>"><?php _e('Remove'); ?></a> <span class="meta-sep"> | </span> <a class="item-cancel submitcancel" id="cancel-<?php echo $item_id; ?>" href="<?php echo esc_url( add_query_arg( array('edit-menu-item' => $item_id, 'cancel' => time()), remove_query_arg( $removed_args, admin_url( 'nav-menus.php' ) ) ) );
+                    ); ?>"><?php _e( ) ) );
                         ?>#menu-item-settings-<?php echo $item_id; ?>"><?php _e('Cancel'); ?></a>
                 </div>
@@ -383 +387 @@
 function wp_nav_menu_setup() {
     // Register meta boxes
-    if ( wp_get_nav_menus() )
-        add_meta_box( 'nav-menu-theme-locations', __( 'Theme Locations' ), 'wp_nav_menu_locations_meta_box' , 'nav-menus', 'side', 'default' );
-    add_meta_box( 'add-custom-links', __('Custom Links'), 'wp_nav_menu_item_link_meta_box', 'nav-menus', 'side', 'default' );
     wp_nav_menu_post_type_meta_boxes();
+
     wp_nav_menu_taxonomy_meta_boxes();
 
@@ -412 +414 @@
         return;
 
-    $initial_meta_boxes = array( 'nav-menu-theme-locations', 'add-custom-links', 'add-page', 'add-category' );
+    $initial_meta_boxes = array( 'nav-menu-theme-locations', 'add-', 'add-category' );
     $hidden_meta_boxes = array();
 
@@ -490 +492 @@
     $num_locations = count( array_keys($locations) );
 
-    echo '<p class="howto">' . sprintf( _n('Your theme supports %s menu. Select which menu you would like to use.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations ), number_format_i18n($num_locations) ) . '</p>';
+    echo '<p class="howto">' .  ) . '</p>';
 
     foreach ( $locations as $location => $description ) {
@@ -513 +515 @@
     ?>
     <p class="button-controls">
-        <?php submit_button( __( 'Save' ), 'primary right', 'nav-menu-locations', false, disabled( $nav_menu_selected_id, 0, false ) ); ?>
+        <?php submit_button( __( 'Save' ), 'primary right', 'nav-menu-locations', false,  ) ); ?>
         <span class="spinner"></span>
     </p>
     <?php
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }
 
@@ -555 +577 @@
             <p id="menu-item-name-wrap">
                 <label class="howto" for="custom-menu-item-name">
-                    <span><?php _e('Label'); ?></span>
+                    <span><?php _e(); ?></span>
                     <input id="custom-menu-item-name" name="menu-item[<?php echo $_nav_menu_placeholder; ?>][menu-item-title]" type="text" class="regular-text menu-item-textbox input-with-default-title" title="<?php esc_attr_e('Menu Item'); ?>" />
                 </label>
@@ -562 +584 @@
         <p class="button-controls">
             <span class="add-to-menu">
-                <input type="submit"<?php disabled( $nav_menu_selected_id, 0 ); ?> class="button-secondary submit-add-to-menu right" value="<?php esc_attr_e('Add to Menu'); ?>" name="add-custom-menu-item" id="submit-customlinkdiv" />
+                <input type="submit"<?php  ); ?> class="button-secondary submit-add-to-menu right" value="<?php esc_attr_e('Add to Menu'); ?>" name="add-custom-menu-item" id="submit-customlinkdiv" />
                 <span class="spinner"></span>
             </span>
@@ -769 +791 @@
             <span class="list-controls">
                 <a href="<?php
-                    echo esc_url(add_query_arg(
+                    echo esc_url(add_query_arg(
                         array(
                             $post_type_name . '-tab' => 'all',
                             'selectall' => 1,
                         ),
-                        remove_query_arg($removed_args)
+                        remove_query_arg()
                     ));
                 ?>#posttype-<?php echo $post_type_name; ?>" class="select-all"><?php _e('Select All'); ?></a>
@@ -780 +802 @@
 
             <span class="add-to-menu">
-                <input type="submit"<?php disabled( $nav_menu_selected_id, 0 ); ?> class="button-secondary submit-add-to-menu right" value="<?php esc_attr_e('Add to Menu'); ?>" name="add-post-type-menu-item" id="submit-posttype-<?php echo $post_type_name; ?>" />
+                <input type="submit"<?php ; ?>" />
                 <span class="spinner"></span>
             </span>
@@ -956 +978 @@
 
             <span class="add-to-menu">
-                <input type="submit"<?php disabled( $nav_menu_selected_id, 0 ); ?> class="button-secondary submit-add-to-menu right" value="<?php esc_attr_e('Add to Menu'); ?>" name="add-taxonomy-menu-item" id="submit-taxonomy-<?php echo $taxonomy_name; ?>" />
+                <input type="submit"<?php ; ?>" />
                 <span class="spinner"></span>
             </span>
@@ -1087 +1109 @@
         $result = '<div id="menu-instructions" class="post-body-plain';
         $result .= ( ! empty($menu_items) ) ? ' menu-instructions-inactive">' : '">';
-        $result .= '<p>' . __('Select menu items (pages, categories, links) from the boxes at left to begin building your custom menu.') . '</p>';
+        $result .= '<p>' . __() . '</p>';
         $result .= '</div>';
 
@@ -1159 +1181 @@
         wp_delete_post( $menu_item_id, true );
 }
-
 add_action('admin_head-nav-menus.php', '_wp_delete_orphaned_draft_menu_items');
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+

trunk/wp-admin/js/nav-menu.js

@@ -46 +46 @@
                 this.initSortables();
 
+
+
+
+
+
+
+
             this.initToggles();
-
-            this.initTabManager();
         },
 
@@ -85 +90 @@
                     return result;
                 },
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
                 updateParentMenuItemDBId : function() {
                     return this.each(function(){
                         var item = $(this),
-                            input = item.find('.menu-item-data-parent-id'),
-                            depth = item.menuItemDepth(),
-                            parent = item.prev();
-
-                        if( depth == 0 ) { // Item is on the top level, has no parent
+                            input = item.find( '.menu-item-data-parent-id' ),
+                            depth = parseInt( item.menuItemDepth() ),
+                            parentDepth = depth - 1,
+                            parent = item.prevAll( '.menu-item-depth-' + parentDepth ).first();
+
+                        if ( 0 == depth ) { // Item is on the top level, has no parent
                             input.val(0);
                         } else { // Find the parent item, and retrieve its object id.
-                            while( ! parent[0] || ! parent[0].className || -1 == parent[0].className.indexOf('menu-item') || ( parent.menuItemDepth() != depth - 1 ) )
-                                parent = parent.prev();
-                            input.val( parent.find('.menu-item-data-db-id').val() );
+                            input.val( parent.find( '.menu-item-data-db-id' ).val() );
                         }
                     });
@@ -121 +162 @@
                     return this.each(function() {
                         var t = $(this), menuItems = {},
-                            checkboxes = t.find('.tabs-panel-active .categorychecklist li input:checked'),
+                            checkboxes = t.find('.tabs-panel-active .categorychecklist li input:checked'),
                             re = new RegExp('menu-item\\[(\[^\\]\]*)');
 
@@ -224 +265 @@
         },
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
         initToggles : function() {
             // init postboxes
@@ -247 +410 @@
                 menuMaxDepth = initialMenuMaxDepth();
 
+
+
+
             // Use the right edge if RTL.
             menuEdge += api.isRTL ? api.menuList.width() : 0;
@@ -309 +475 @@
                     children = transport.children().insertAfter(ui.item);
 
+
+
+
+
+
+
+
                     // Update depth classes
                     if( depthChange != 0 ) {
@@ -328 +501 @@
                         ui.item[0].style.right = 0;
                     }
-
-                    // The width of the tab bar might have changed. Just in case.
-                    api.refreshMenuTabs( true );
                 },
                 change: function(e, ui) {
@@ -462 +632 @@
                     $t.addClass( name ).val( $t.data(name) );
             });
+
+
         },
 
@@ -573 +745 @@
                 var ins = $('#menu-instructions');
                 processMethod(menuMarkup, params);
-                if( ! ins.hasClass('menu-instructions-inactive') && ins.siblings().length )
-                    ins.addClass('menu-instructions-inactive');
+                // Make it stand out a bit more visually, by adding a fadeIn
+                $( 'li.pending' ).hide().fadeIn('slow');
+                $( '.drag-instructions' ).show();
+                if( ! ins.hasClass( 'menu-instructions-inactive' ) && ins.siblings().length )
+                    ins.addClass( 'menu-instructions-inactive' );
                 callback();
             });
@@ -587 +762 @@
         addMenuItemToBottom : function( menuMarkup, req ) {
             $(menuMarkup).hideAdvancedMenuItemFields().appendTo( api.targetList );
+
         },
 
         addMenuItemToTop : function( menuMarkup, req ) {
             $(menuMarkup).hideAdvancedMenuItemFields().prependTo( api.targetList );
+
         },
 
@@ -605 +782 @@
             } else {
                 // Make the post boxes read-only, as they can't be used yet
-                $('#menu-settings-column').find('input,select').prop('disabled', true).end().find('a').attr('href', '#').unbind('click');
+                $();
             }
         },
@@ -686 +863 @@
                     return false;
                 }
-            });
-        },
-
-        initTabManager : function() {
-            var fixed = $('.nav-tabs-wrapper'),
-                fluid = fixed.children('.nav-tabs'),
-                active = fluid.children('.nav-tab-active'),
-                tabs = fluid.children('.nav-tab'),
-                tabsWidth = 0,
-                fixedRight, fixedLeft,
-                arrowLeft, arrowRight, resizeTimer, css = {},
-                marginFluid = api.isRTL ? 'margin-right' : 'margin-left',
-                marginFixed = api.isRTL ? 'margin-left' : 'margin-right',
-                msPerPx = 2;
-
-            /**
-             * Refreshes the menu tabs.
-             * Will show and hide arrows where necessary.
-             * Scrolls to the active tab by default.
-             *
-             * @param savePosition {boolean} Optional. Prevents scrolling so
-             *        that the current position is maintained. Default false.
-             **/
-            api.refreshMenuTabs = function( savePosition ) {
-                var fixedWidth = fixed.width(),
-                    margin = 0, css = {};
-                fixedLeft = fixed.offset().left;
-                fixedRight = fixedLeft + fixedWidth;
-
-                if( !savePosition )
-                    active.makeTabVisible();
-
-                // Prevent space from building up next to the last tab if there's more to show
-                if( tabs.last().isTabVisible() ) {
-                    margin = fixed.width() - tabsWidth;
-                    margin = margin > 0 ? 0 : margin;
-                    css[marginFluid] = margin + 'px';
-                    fluid.animate( css, 100, "linear" );
-                }
-
-                // Show the arrows only when necessary
-                if( fixedWidth > tabsWidth )
-                    arrowLeft.add( arrowRight ).hide();
-                else
-                    arrowLeft.add( arrowRight ).show();
-            }
-
-            $.fn.extend({
-                makeTabVisible : function() {
-                    var t = this.eq(0), left, right, css = {}, shift = 0;
-
-                    if( ! t.length ) return this;
-
-                    left = t.offset().left;
-                    right = left + t.outerWidth();
-
-                    if( right > fixedRight )
-                        shift = fixedRight - right;
-                    else if ( left < fixedLeft )
-                        shift = fixedLeft - left;
-
-                    if( ! shift ) return this;
-
-                    css[marginFluid] = "+=" + api.negateIfRTL * shift + 'px';
-                    fluid.animate( css, Math.abs( shift ) * msPerPx, "linear" );
-                    return this;
-                },
-                isTabVisible : function() {
-                    var t = this.eq(0),
-                        left = t.offset().left,
-                        right = left + t.outerWidth();
-                    return ( right <= fixedRight && left >= fixedLeft ) ? true : false;
-                }
-            });
-
-            // Find the width of all tabs
-            tabs.each(function(){
-                tabsWidth += $(this).outerWidth(true);
-            });
-
-            // Set up fixed margin for overflow, unset padding
-            css['padding'] = 0;
-            css[marginFixed] = (-1 * tabsWidth) + 'px';
-            fluid.css( css );
-
-            // Build tab navigation
-            arrowLeft = $('<div class="nav-tabs-arrow nav-tabs-arrow-left"><a>&laquo;</a></div>');
-            arrowRight = $('<div class="nav-tabs-arrow nav-tabs-arrow-right"><a>&raquo;</a></div>');
-            // Attach to the document
-            fixed.wrap('<div class="nav-tabs-nav"/>').parent().prepend( arrowLeft ).append( arrowRight );
-
-            // Set the menu tabs
-            api.refreshMenuTabs();
-            // Make sure the tabs reset on resize
-            $(window).resize(function() {
-                if( resizeTimer ) clearTimeout(resizeTimer);
-                resizeTimer = setTimeout( api.refreshMenuTabs, 200);
-            });
-
-            // Build arrow functions
-            $.each([{
-                    arrow : arrowLeft,
-                    next : "next",
-                    last : "first",
-                    operator : "+="
-                },{
-                    arrow : arrowRight,
-                    next : "prev",
-                    last : "last",
-                    operator : "-="
-                }], function(){
-                var that = this;
-                this.arrow.mousedown(function(){
-                    var marginFluidVal = Math.abs( parseInt( fluid.css(marginFluid) ) ),
-                        shift = marginFluidVal,
-                        css = {};
-
-                    if( "-=" == that.operator )
-                        shift = Math.abs( tabsWidth - fixed.width() ) - marginFluidVal;
-
-                    if( ! shift ) return;
-
-                    css[marginFluid] = that.operator + shift + 'px';
-                    fluid.animate( css, shift * msPerPx, "linear" );
-                }).mouseup(function(){
-                    var tab, next;
-                    fluid.stop(true);
-                    tab = tabs[that.last]();
-                    while( (next = tab[that.next]()) && next.length && ! next.isTabVisible() ) {
-                        tab = next;
-                    }
-                    tab.makeTabVisible();
-                });
             });
         },
@@ -847 +891 @@
 
         eventOnClickCancelLink : function(clickedEl) {
-            var settings = $(clickedEl).closest('.menu-item-settings');
-            settings.setItemData( settings.data('menu-item-data') );
+            var settings = $( clickedEl ).closest( '.menu-item-settings' ),
+                thisMenuItem = $( clickedEl ).closest( '.menu-item' );
+            thisMenuItem.removeClass('menu-item-edit-active').addClass('menu-item-edit-inactive');
+            settings.setItemData( settings.data('menu-item-data') ).hide();
             return false;
         },
@@ -945 +991 @@
                     var ins = $('#menu-instructions');
                     el.remove();
-                    children.shiftDepthClass(-1).updateParentMenuItemDBId();
-                    if( ! ins.siblings().length )
-                        ins.removeClass('menu-instructions-inactive');
+                    children.shiftDepthClass( -1 ).updateParentMenuItemDBId();
+                    if( 0 == $( '#menu-to-edit li' ).length ) {
+                        $( '.drag-instructions' ).hide();
+                        ins.removeClass( 'menu-instructions-inactive' );
+                    }
                 });
         },

trunk/wp-admin/nav-menus.php

@@ -222 +222 @@
             $messages[] = '<div id="message" class="updated"><p>' . __('The menu item has been successfully deleted.') . '</p></div>';
         break;
+
     case 'delete':
         check_admin_referer( 'delete-nav_menu-' . $nav_menu_selected_id );
-
         if ( is_nav_menu( $nav_menu_selected_id ) ) {
-            $deleted_nav_menu = wp_get_nav_menu_object( $nav_menu_selected_id );
-            $delete_nav_menu = wp_delete_nav_menu( $nav_menu_selected_id );
-
-            if ( is_wp_error($delete_nav_menu) ) {
-                $messages[] = '<div id="message" class="error"><p>' . $delete_nav_menu->get_error_message() . '</p></div>';
-            } else {
-                // Remove this menu from any locations.
-                $locations = get_theme_mod( 'nav_menu_locations' );
-                foreach ( (array) $locations as $location => $menu_id ) {
-                    if ( $menu_id == $nav_menu_selected_id )
-                        $locations[ $location ] = 0;
-                }
-                set_theme_mod( 'nav_menu_locations', $locations );
-                $messages[] = '<div id="message" class="updated"><p>' . __('The menu has been successfully deleted.') . '</p></div>';
-                // Select the next available menu
-                $nav_menu_selected_id = 0;
-                $_nav_menus = wp_get_nav_menus( array('orderby' => 'name') );
-                foreach( $_nav_menus as $index => $_nav_menu ) {
-                    if ( strcmp( $_nav_menu->name, $deleted_nav_menu->name ) >= 0
-                     || $index == count( $_nav_menus ) - 1 ) {
-                        $nav_menu_selected_id = $_nav_menu->term_id;
-                        break;
-                    }
-                }
-            }
-            unset( $delete_nav_menu, $deleted_nav_menu, $_nav_menus );
+            $deletion = _wp_delete_nav_menu( $nav_menu_selected_id );
         } else {
             // Reset the selected menu
@@ -257 +232 @@
             unset( $_REQUEST['menu'] );
         }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
         break;
 
@@ -262 +262 @@
         check_admin_referer( 'update-nav_menu', 'update-nav-menu-nonce' );
 
-        // Update menu theme locations
-        if ( isset( $_POST['menu-locations'] ) )
-            set_theme_mod( 'nav_menu_locations', array_map( 'absint', $_POST['menu-locations'] ) );
+        // Get existing menu locations assignments
+        $locations = get_registered_nav_menus();
+        $menu_locations = get_nav_menu_locations();
+
+        // Remove menu locations that have been unchecked
+        foreach ( $locations as $location => $description ) {
+            if ( ( empty( $_POST['menu-locations'] ) || empty( $_POST['menu-locations'][ $location ] ) ) && isset( $menu_locations[ $location ] ) && $menu_locations[ $location ] == $nav_menu_selected_id )
+                unset( $menu_locations[ $location ] );
+        }
+
+        // Merge new and existing menu locations if any new ones are set
+        if ( isset( $_POST['menu-locations'] ) ) {
+            $new_menu_locations = array_map( 'absint', $_POST['menu-locations'] );
+            $menu_locations = array_merge( $menu_locations, $new_menu_locations );
+        }
+
+        // Set menu locations
+        set_theme_mod( 'nav_menu_locations', $menu_locations );
 
         // Add Menu
@@ -279 +294 @@
                     $nav_menu_selected_id = $_nav_menu_selected_id;
                     $nav_menu_selected_title = $_menu_object->name;
-                    $messages[] = '<div id="message" class="updated"><p>' . sprintf( __('The <strong>%s</strong> menu has been successfully created.'), $nav_menu_selected_title ) . '</p></div>';
+                    if ( isset( $_REQUEST['menu-item'] ) )
+                        wp_save_nav_menu_items( $nav_menu_selected_id, absint( $_REQUEST['menu-item'] ) );
+                    if ( isset( $_REQUEST['zero-menu-state'] ) ) {
+                        // If there are menu items, add them
+                        wp_nav_menu_update_menu_items( $nav_menu_selected_id, $nav_menu_selected_title );
+                        // Auto-save nav_menu_locations
+                        $locations = get_theme_mod( 'nav_menu_locations' );
+                        foreach ( (array) $locations as $location => $menu_id ) {
+                                $locations[ $location ] = $nav_menu_selected_id;
+                                break; // There should only be 1
+                        }
+                        set_theme_mod( 'nav_menu_locations', $locations );
+                    }
+                    $messages[] = '<div id="message" class="updated"><p>' . sprintf( __( '<strong>%s</strong> has been created.' ), $nav_menu_selected_title ) . '</p></div>';
                 }
             } else {
-                $messages[] = '<div id="message" class="error"><p>' . __('Please enter a valid menu name.') . '</p></div>';
+                $messages[] = '<div id="message" class="error"><p>' . __() . '</p></div>';
             }
 
-        // update existing menu
+        // pdate existing menu
         } else {
 
@@ -292 +320 @@
             $menu_title = trim( esc_html( $_POST['menu-name'] ) );
             if ( ! $menu_title ) {
-                $messages[] = '<div id="message" class="error"><p>' . __('Please enter a valid menu name.') . '</p></div>';
+                $messages[] = '<div id="message" class="error"><p>' . __() . '</p></div>';
                 $menu_title = $_menu_object->name;
             }
@@ -308 +336 @@
 
             // Update menu items
-
             if ( ! is_wp_error( $_menu_object ) ) {
-                $unsorted_menu_items = wp_get_nav_menu_items( $nav_menu_selected_id, array('orderby' => 'ID', 'output' => ARRAY_A, 'output_key' => 'ID', 'post_status' => 'draft,publish') );
-                $menu_items = array();
-                // Index menu items by db ID
-                foreach( $unsorted_menu_items as $_item )
-                    $menu_items[$_item->db_id] = $_item;
-
-                $post_fields = array( 'menu-item-db-id', 'menu-item-object-id', 'menu-item-object', 'menu-item-parent-id', 'menu-item-position', 'menu-item-type', 'menu-item-title', 'menu-item-url', 'menu-item-description', 'menu-item-attr-title', 'menu-item-target', 'menu-item-classes', 'menu-item-xfn' );
-                wp_defer_term_counting(true);
-                // Loop through all the menu items' POST variables
-                if ( ! empty( $_POST['menu-item-db-id'] ) ) {
-                    foreach( (array) $_POST['menu-item-db-id'] as $_key => $k ) {
-
-                        // Menu item title can't be blank
-                        if ( empty( $_POST['menu-item-title'][$_key] ) )
-                            continue;
-
-                        $args = array();
-                        foreach ( $post_fields as $field )
-                            $args[$field] = isset( $_POST[$field][$_key] ) ? $_POST[$field][$_key] : '';
-
-                        $menu_item_db_id = wp_update_nav_menu_item( $nav_menu_selected_id, ( $_POST['menu-item-db-id'][$_key] != $_key ? 0 : $_key ), $args );
-
-                        if ( is_wp_error( $menu_item_db_id ) )
-                            $messages[] = '<div id="message" class="error"><p>' . $menu_item_db_id->get_error_message() . '</p></div>';
-                        elseif ( isset( $menu_items[$menu_item_db_id] ) )
-                            unset( $menu_items[$menu_item_db_id] );
-                    }
-                }
-
-                // Remove menu items from the menu that weren't in $_POST
-                if ( ! empty( $menu_items ) ) {
-                    foreach ( array_keys( $menu_items ) as $menu_item_id ) {
-                        if ( is_nav_menu_item( $menu_item_id ) ) {
-                            wp_delete_post( $menu_item_id );
-                        }
-                    }
-                }
-
-                // Store 'auto-add' pages.
-                $auto_add = ! empty( $_POST['auto-add-pages'] );
-                $nav_menu_option = (array) get_option( 'nav_menu_options' );
-                if ( ! isset( $nav_menu_option['auto_add'] ) )
-                    $nav_menu_option['auto_add'] = array();
-                if ( $auto_add ) {
-                    if ( ! in_array( $nav_menu_selected_id, $nav_menu_option['auto_add'] ) )
-                        $nav_menu_option['auto_add'][] = $nav_menu_selected_id;
-                } else {
-                    if ( false !== ( $key = array_search( $nav_menu_selected_id, $nav_menu_option['auto_add'] ) ) )
-                        unset( $nav_menu_option['auto_add'][$key] );
-                }
-                // Remove nonexistent/deleted menus
-                $nav_menu_option['auto_add'] = array_intersect( $nav_menu_option['auto_add'], wp_get_nav_menus( array( 'fields' => 'ids' ) ) );
-                update_option( 'nav_menu_options', $nav_menu_option );
-
-                wp_defer_term_counting(false);
-
-                do_action( 'wp_update_nav_menu', $nav_menu_selected_id );
-
-                $messages[] = '<div id="message" class="updated"><p>' . sprintf( __('The <strong>%s</strong> menu has been updated.'), $nav_menu_selected_title ) . '</p></div>';
-                unset( $menu_items, $unsorted_menu_items );
+                $messages = array_merge( $messages, wp_nav_menu_update_menu_items( $nav_menu_selected_id, $nav_menu_selected_title ) );
             }
         }
@@ -377 +345 @@
 // Get all nav menus
 $nav_menus = wp_get_nav_menus( array('orderby' => 'name') );
+
+
+
+
+
+
+
+
+
+
+
+
 
 // Get recently edited nav menu
-$recently_edited = (int) get_user_option( 'nav_menu_recently_edited' );
-
-// If there was no recently edited menu, and $nav_menu_selected_id is a nav menu, update recently edited menu.
-if ( !$recently_edited && is_nav_menu( $nav_menu_selected_id ) ) {
+$recently_edited = absint( get_user_option( 'nav_menu_recently_edited' ) );
+if ( empty( $recently_edited ) && is_nav_menu( $nav_menu_selected_id ) )
     $recently_edited = $nav_menu_selected_id;
 
-// Else if $nav_menu_selected_id is not a menu and not requesting that we create a new menu, but $recently_edited is a menu, grab that one.
-} elseif ( 0 == $nav_menu_selected_id && ! isset( $_REQUEST['menu'] ) && is_nav_menu( $recently_edited ) ) {
+// 
+if ( empty( $nav_menu_selected_id ) && ! isset( $_GET['menu'] ) && is_nav_menu( $recently_edited ) )
     $nav_menu_selected_id = $recently_edited;
 
-// Else try to grab the first menu from the menus list
-} elseif ( 0 == $nav_menu_selected_id && ! isset( $_REQUEST['menu'] ) && ! empty($nav_menus) ) {
+// t
+if ( ! $add_new_screen && 0 < $menu_count && isset( $_GET['action'] ) && 'delete' == $_GET['action'] )
     $nav_menu_selected_id = $nav_menus[0]->term_id;
-}
+
+// Set $nav_menu_selected_id to 0 if no menus
+if ( $one_theme_location_no_menus )
+    $nav_menu_selected_id = 0;
 
 // Update the user's setting
@@ -413 +394 @@
 }
 
+
+
+
+
+
+
 // Ensure the user will be able to scroll horizontally
 // by adding a class for the max menu depth.
@@ -419 +406 @@
 
 // Calling wp_get_nav_menu_to_edit generates $_wp_nav_menu_max_depth
-if ( is_nav_menu( $nav_menu_selected_id ) )
+if ( is_nav_menu( $nav_menu_selected_id ) ) {
+    $menu_items = wp_get_nav_menu_items( $nav_menu_selected_id, array( 'post_status' => 'any' ) );
     $edit_markup = wp_get_nav_menu_to_edit( $nav_menu_selected_id );
+
 
 function wp_nav_menu_max_depth($classes) {
@@ -461 +450 @@
 <div class="wrap">
     <?php screen_icon(); ?>
-    <h2><?php esc_html_e('Menus'); ?></h2>
+    <h2><?php ></h2>
     <?php
     foreach( $messages as $message ) :
@@ -467 +456 @@
     endforeach;
     ?>
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
     <div id="nav-menus-frame">
-    <div id="menu-settings-column" class="metabox-holder<?php if ( !$nav_menu_selected_id ) { echo ' metabox-holder-disabled'; } ?>">
+    <div id="menu-settings-column" class="metabox-holder<?php if ( isset( $_GET['menu'] ) && '0' == $_GET['menu'] ) { echo ' metabox-holder-disabled'; } ?>">
+
+        <div class="clear"></div>
 
         <form id="nav-menu-meta" action="<?php echo admin_url( 'nav-menus.php' ); ?>" class="nav-menu-meta" method="post" enctype="multipart/form-data">
@@ -480 +506 @@
     <div id="menu-management-liquid">
         <div id="menu-management">
-            <div id="select-nav-menu-container" class="hide-if-js">
-                <form id="select-nav-menu" action="">
-                    <strong><label for="select-nav-menu"><?php esc_html_e( 'Select Menu:' ); ?></label></strong>
-                    <select class="select-nav-menu" name="menu">
-                        <?php foreach( (array) $nav_menus as $_nav_menu ) : ?>
-                            <option value="<?php echo esc_attr($_nav_menu->term_id) ?>" <?php selected($nav_menu_selected_id, $_nav_menu->term_id); ?>>
-                                <?php echo esc_html( $_nav_menu->truncated_name ); ?>
-                            </option>
-                        <?php endforeach; ?>
-                        <option value="0"><?php esc_html_e('Add New Menu'); ?></option>
-                    </select>
-                    <input type="hidden" name="action" value="edit" />
-                    <?php submit_button( __( 'Select' ), 'secondary', 'select_menu', false ); ?>
-                </form>
-            </div>
-            <div class="nav-tabs-wrapper">
-            <div class="nav-tabs">
-                <?php
-                foreach( (array) $nav_menus as $_nav_menu ) :
-                    if ( $nav_menu_selected_id == $_nav_menu->term_id ) : ?><span class="nav-tab nav-tab-active">
-                            <?php echo esc_html( $_nav_menu->truncated_name ); ?>
-                        </span><?php else : ?><a href="<?php
-                            echo esc_url(add_query_arg(
-                                array(
-                                    'action' => 'edit',
-                                    'menu' => $_nav_menu->term_id,
-                                ),
-                                admin_url( 'nav-menus.php' )
-                            ));
-                        ?>" class="nav-tab hide-if-no-js">
-                            <?php echo esc_html( $_nav_menu->truncated_name ); ?>
-                        </a><?php endif;
-                endforeach;
-                if ( 0 == $nav_menu_selected_id ) : ?><span class="nav-tab menu-add-new nav-tab-active">
-                    <?php printf( '<abbr title="%s">+</abbr>', esc_html__( 'Add menu' ) ); ?>
-                </span><?php else : ?><a href="<?php
-                    echo esc_url(add_query_arg(
-                        array(
-                            'action' => 'edit',
-                            'menu' => 0,
-                        ),
-                        admin_url( 'nav-menus.php' )
-                    ));
-                ?>" class="nav-tab menu-add-new">
-                    <?php printf( '<abbr title="%s">+</abbr>', esc_html__( 'Add menu' ) ); ?>
-                </a><?php endif; ?>
-            </div>
-            </div>
-            <div class="menu-edit">
-                <form id="update-nav-menu" action="<?php echo admin_url( 'nav-menus.php' ); ?>" method="post" enctype="multipart/form-data">
+            <form id="update-nav-menu" action="<?php echo esc_url( admin_url( 'nav-menus.php' ) ); ?>" method="post" enctype="multipart/form-data">
+                <div class="menu-edit <?php if ( $add_new_screen ) echo 'blank-slate'; ?>">
+                    <?php
+                    wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false );
+                    wp_nonce_field( 'meta-box-order', 'meta-box-order-nonce', false );
+                    wp_nonce_field( 'update-nav_menu', 'update-nav-menu-nonce' );
+
+                    if ( $one_theme_location_no_menus ) { ?>
+                        <input type="hidden" name="zero-menu-state" value="true" />
+                    <?php } ?>
+                    <input type="hidden" name="action" value="update" />
+                    <input type="hidden" name="menu" id="menu" value="<?php echo esc_attr( $nav_menu_selected_id ); ?>" />
                     <div id="nav-menu-header">
-                        <div id="submitpost" class="submitbox">
-                            <div class="major-publishing-actions">
-                                <label class="menu-name-label howto open-label" for="menu-name">
-                                    <span><?php _e('Menu Name'); ?></span>
-                                    <input name="menu-name" id="menu-name" type="text" class="menu-name regular-text menu-item-textbox input-with-default-title" title="<?php esc_attr_e('Enter menu name here'); ?>" value="<?php echo esc_attr( $nav_menu_selected_title ); ?>" />
-                                </label>
-                                <?php if ( !empty( $nav_menu_selected_id ) ) :
-                                    if ( ! isset( $auto_add ) ) {
-                                        $auto_add = get_option( 'nav_menu_options' );
-                                        if ( ! isset( $auto_add['auto_add'] ) )
-                                            $auto_add = false;
-                                        elseif ( false !== array_search( $nav_menu_selected_id, $auto_add['auto_add'] ) )
-                                            $auto_add = true;
-                                        else
-                                            $auto_add = false;
-                                    }
-                                ?>
-                                <div class="auto-add-pages">
-                                    <label class="howto"><input type="checkbox"<?php checked( $auto_add ); ?> name="auto-add-pages" value="1" /> <?php printf( __('Automatically add new top-level pages' ), esc_url( admin_url( 'edit.php?post_type=page' ) ) ); ?></label>
-                                </div>
-                                <?php endif; ?>
-                                <br class="clear" />
-                                <div class="publishing-action">
-                                    <?php submit_button( empty( $nav_menu_selected_id ) ? __( 'Create Menu' ) : __( 'Save Menu' ), 'button-primary menu-save', 'save_menu', false, array( 'id' => 'save_menu_header' ) ); ?>
-                                </div><!-- END .publishing-action -->
-
-                                <?php if ( ! empty( $nav_menu_selected_id ) ) : ?>
-                                <div class="delete-action">
-                                    <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>
-                                </div><!-- END .delete-action -->
-                                <?php endif; ?>
-                            </div><!-- END .major-publishing-actions -->
-                        </div><!-- END #submitpost .submitbox -->
-                        <?php
-                        wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false );
-                        wp_nonce_field( 'meta-box-order', 'meta-box-order-nonce', false );
-                        wp_nonce_field( 'update-nav_menu', 'update-nav-menu-nonce' );
-                        ?>
-                        <input type="hidden" name="action" value="update" />
-                        <input type="hidden" name="menu" id="menu" value="<?php echo esc_attr( $nav_menu_selected_id ); ?>" />
-                    </div><!-- END #nav-menu-header -->
+                        <div class="major-publishing-actions">
+                            <label class="menu-name-label howto open-label" for="menu-name">
+                                <span><?php _e( 'Menu Name' ); ?></span>
+                                <input name="menu-name" id="menu-name" type="text" class="menu-name regular-text menu-item-textbox input-with-default-title" title="<?php esc_attr_e( 'Enter menu name here' ); ?>" value="<?php if ( $one_theme_location_no_menus ) _e( 'Menu 1' ); else echo esc_attr( $nav_menu_selected_title ); ?>" />
+                            </label>
+                            <div class="publishing-action">
+                                <?php submit_button( empty( $nav_menu_selected_id ) ? __( 'Create Menu' ) : __( 'Save Menu' ), 'button-primary menu-save', 'save_menu', false, array( 'id' => 'save_menu_header' ) ); ?>
+                            </div><!-- END .publishing-action -->
+                        </div><!-- END .major-publishing-actions -->
+                    </div><!-- END .nav-menu-header -->
                     <div id="post-body">
                         <div id="post-body-content">
+
+
+
+
+
                             <?php
-                            if ( isset( $edit_markup ) ) {
-                                if ( ! is_wp_error( $edit_markup ) )
-                                    echo $edit_markup;
-                            } else if ( empty( $nav_menu_selected_id ) ) {
-                                echo '<div class="post-body-plain">';
-                                echo '<p>' . __('To create a custom menu, give it a name above and click Create Menu. Then choose items like pages, categories or custom links from the left column to add to this menu.') . '</p>';
-                                echo '<p>' . __('After you have added your items, drag and drop to put them in the order you want. You can also click each item to reveal additional configuration options.') . '</p>';
-                                echo '<p>' . __('When you have finished building your custom menu, make sure you click the Save Menu button.') . '</p>';
-                                echo '</div>';
-                            }
+                            if ( isset( $edit_markup ) && ! is_wp_error( $edit_markup ) ) {
+                                echo $edit_markup;
+                            } else {
                             ?>
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
                         </div><!-- /#post-body-content -->
                     </div><!-- /#post-body -->
                     <div id="nav-menu-footer">
                         <div class="major-publishing-actions">
-                        <div class="publishing-action">
-                            <?php
-                            if ( ! empty( $nav_menu_selected_id ) )
-                                submit_button( __( 'Save Menu' ), 'button-primary menu-save', 'save_menu', false, array( 'id' => 'save_menu_footer' ) );
-                            ?>
-                        </div>
-                        </div>
+                            <?php if ( 0 != $menu_count && ! $add_new_screen ) : ?>
+                            <span class="delete-action">
+                                <a class="submitdelete deletion menu-delete" href="<?php echo esc_url( wp_nonce_url( add_query_arg( array( 'action' => 'delete', 'menu' => $nav_menu_selected_id, admin_url() ) ), 'delete-nav_menu-' . $nav_menu_selected_id) ); ?>"><?php _e('Delete Menu'); ?></a>
+                            </span><!-- END .delete-action -->
+                            <?php endif; ?>
+                            <div class="publishing-action">
+                                <?php submit_button( empty( $nav_menu_selected_id ) ? __( 'Create Menu' ) : __( 'Save Menu' ), 'button-primary menu-save', 'save_menu', false, array( 'id' => 'save_menu_header' ) ); ?>
+                            </div><!-- END .publishing-action -->
+                        </div><!-- END .major-publishing-actions -->
                     </div><!-- /#nav-menu-footer -->
-                </form><!-- /#update-nav-menu -->
-            </div><!-- /.menu-edit -->
+                </ -->
+            </ -->
         </div><!-- /#menu-management -->
     </div><!-- /#menu-management-liquid -->
     </div><!-- /#nav-menus-frame -->
 </div><!-- /.wrap-->
-
+<script type="text/javascript">var oneThemeLocationNoMenus = <?php if ( $one_theme_location_no_menus ) echo 'true'; else echo 'false'; ?>;</script>
 <?php include( './admin-footer.php' ); ?>