[wp-trac] [WordPress Trac] #11817: Better Site Menu Management

WordPress Trac wp-trac at lists.automattic.com
Thu Mar 18 03:48:06 UTC 2010


#11817: Better Site Menu Management
-------------------------+--------------------------------------------------
 Reporter:  scribu       |        Owner:          
     Type:  enhancement  |       Status:  reopened
 Priority:  normal       |    Milestone:  3.0     
Component:  General      |      Version:          
 Severity:  normal       |   Resolution:          
 Keywords:  has-patch    |  
-------------------------+--------------------------------------------------

Comment(by ceenz):

 Replying to [comment:332 nathanrice]:
 > Replying to [comment:331 ceenz]:
 > > Each instance of nav_menu should have an id, name, array for items and
 array for options.
 > > Each nav_menu_item should have an id, name, link, type, and array for
 options.
 > >
 > > That is seriously all that is required.
 >
 > I don't think that's true. For starters, the classes that get applied to
 menu items are heavily dependent on data relationships and pageview
 context.

 Menu and menu item classes should be applied at display runtime based on
 pageview context etc using display menu hook and extenable php class, not
 stored in DB.


 >
 > > All the BS surrounding custom post types different taxonomy (eg tag,
 categories) can be handled through an extend-able nav_menu class based on
 the nav_menu_item type and store the unique options for the type in the
 nav_menu_item options array.
 >
 > But why? If WordPress is going to offer menu management AND custom post-
 types and taxonomies, then why would the two not have some sort of
 "official" relationship?

 Because it is really only adding greater complexity to an already complex
 system for VERY LITTLE gain. Taxonomy is a descriptor of data (eg type of
 category for a post). A menu does not describe data in any useful way and
 therefore should not be stored as taxonomy.
 >
 > > I am really sorry but the whole situation is making me really angry.
 >
 > No kidding.
 :-)

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11817#comment:333>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list