[wp-trac] [WordPress Trac] #14134: Menus item are limited to 16 item and will not save more than that

WordPress Trac noreply at wordpress.org
Fri Jul 18 00:02:19 UTC 2014


#14134: Menus item are limited to 16 item and will not save more than that
-------------------------------------------------+-------------------------
 Reporter:  jaanfx                               |       Owner:  filosofo
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  high                                 |   Milestone:  Future
Component:  Menus                                |  Release
 Severity:  major                                |     Version:  3.0
 Keywords:  has-patch needs-refresh needs-       |  Resolution:
  testing                                        |     Focuses:
-------------------------------------------------+-------------------------
Changes (by celloexpressions):

 * milestone:  4.0 => Future Release


Comment:

 Unfortunately, this is nowhere near being ready for 4.0.

 The only way to fully solve this and make the menus admin fully scalable
 is to save menu items individually via ajax as they're changed. This can
 be done without immediately publishing those changes, but doing so within
 the current framework would be tricky.

 [http://make.wordpress.org/core/2014/07/15/gsoc-menu-customizer-update-
 scalable-menus/ My Menu Customizer GSoC Project now does this,
 implementing Menus in a scalable way]. That project is a few releases from
 being ready from core but will most likely make it into core in some form
 at some point. The plugin cannot be used to work around this bug just yet,
 but it most likely will be able to do so in the next couple months, as the
 internals are cleaned up and its development shifts focus to UI. Since
 that project aims to replace the existing Menus screen entirely (where the
 Customizer is supported), I don't think refactoring the existing Menus
 screen to do things on an item-by-item basis, and using Backbone.js, is a
 good use of time.

 There are some things we can do here to make things work better:
 - [attachment:wordpress-menuitem-3.9.patch​] is a good idea, if it does
 what it says, but it needs extensive review and testing. This can come
 from anyone, at least initially, not just core  committers, so if you'd
 like to help move this ticket please jump in here.
 - `wp_update_nav_menu_items()` is a mess, and is responsible for a lot of
 the issues that trickle down to the UI and this ticket. It could use a
 major overhaul, or, at the very least, #28138.
 - Menu items each have 13 fields that are inputs on nav-menus.php. Several
 hidden inputs have values that never change, and those could be removed if
 we do #28138.


 We could get to a reasonable improvement here with a combination of the
 above, but that'll take a good amount of work from several people
 patching, reviewing, and testing here.

 See also: https://plugins.trac.wordpress.org/changeset/946618/menu-
 customizer

--
Ticket URL: <https://core.trac.wordpress.org/ticket/14134#comment:149>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list