[wp-trac] [WordPress Trac] #23119: UX Improvements to nav-menus.php
WordPress Trac
noreply at wordpress.org
Wed Feb 6 07:04:36 UTC 2013
#23119: UX Improvements to nav-menus.php
-----------------------------------------------+------------------
Reporter: lessbloat | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.6
Component: Menus | Version:
Severity: normal | Resolution:
Keywords: has-patch needs-testing 3.6-menus |
-----------------------------------------------+------------------
Comment (by jkudish):
In [http://core.trac.wordpress.org/attachment/ticket/23119/23119.24.diff
23119.24.diff] I've added support for setting theme locations via
checkboxes under 'Menu Settings'.
Note that this patch is sort of the minimal amount of work/code required
to make this work, it can/needs to be cleaned up a bit and if we proceed
with this approach we also need to remove the theme locations metabox code
(which I've commented out the `add_meta_box` line for now + added a todo
for)
Here's how the checkbox looks:
[[Image(http://core.trac.wordpress.org/raw-
attachment/ticket/23119/Screen%20Shot%202013-02-05%20at%2010.43.19%20PM.png)]]
Here's it looks if a menu is already assigned to a location (adds a note
under the checkbox):
[[Image(http://core.trac.wordpress.org/raw-
attachment/ticket/23119/Screen%20Shot%202013-02-05%20at%2010.43.33%20PM.png)]]
Let me know what you think and if you can test this, please do and let me
know if there's any issues.
I also have 2 unrelated notes of something I noticed with this latest
patch that we should probably fix:
* 'delete menu' link should be red
* there should be some explanatory text under 'Menu Structure' before you
add items to it, otherwise it's a very confusing heading on it's own
--
Ticket URL: <http://core.trac.wordpress.org/ticket/23119#comment:170>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list