[wp-trac] [WordPress Trac] #12812: New Nav Menu needs to have ancestor class

WordPress Trac wp-trac at lists.automattic.com
Mon May 24 15:10:31 UTC 2010


#12812: New Nav Menu needs to have ancestor class
-------------------------+--------------------------------------------------
 Reporter:  joostdevalk  |       Owner:  filosofo   
     Type:  enhancement  |      Status:  new        
 Priority:  normal       |   Milestone:  3.0        
Component:  Menus        |     Version:             
 Severity:  normal       |    Keywords:  needs-patch
-------------------------+--------------------------------------------------

Comment(by husobj):

 Replying to [comment:4 ryan]:
 > These aren't really menu item parents and ancestors. They are page
 parents and ancestors. Menu item hierarchy doesn't have to follow page
 hierarchy. We need to keep in mind the difference.

 Ryan, you have a point there - should a menu ancestor be highlighted if
 the current selected menu item is a child of that menu item (but not
 necessarily a child of that page)?

 ... and should the menu item be highlighted if the current page is a child
 page of that ancestor (but not necessarily a child menu item)

 My opinion is that a menu item should be highlighted if the current page
 is a child of that page OR a child menu item of that page.

 Just my 2 cents :)

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


More information about the wp-trac mailing list