[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