Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#13664 closed defect (bug) (fixed)

Consolidate menu item class assignments

Reported by: nacin's profile nacin Owned by: filosofo's profile filosofo
Milestone: 3.0 Priority: high
Severity: normal Version: 3.0
Component: Menus Keywords: has-patch
Focuses: Cc:

Description

We should be assigning classes to menu items using the helper function _wp_menu_item_classes_by_context(). The classes being assigned in Walker_Nav_Menu::start_el() need to be moved there, including the ones that are back compat for wp_page_menu(). The current divide will cause inconsistencies when custom walkers are used.

Attachments (1)

consolidate-class-calc.13664.diff (6.4 KB) - added by filosofo 15 years ago.

Download all attachments as: .zip

Change History (4)

#1 @filosofo
15 years ago

  • Owner set to filosofo
  • Status changed from new to accepted

#2 @filosofo
15 years ago

  • Keywords has-patch added; needs-patch removed
  • Version set to 3.0

#3 @ryan
15 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

(In [15105]) Consolidate nav menu class assignments. Props filosofo. fixes #13664

Note: See TracTickets for help on using tickets.