Make WordPress Core

Opened 16 years ago

Closed 12 years ago

Last modified 12 years ago

#14687 closed feature request (duplicate)

Add "collapsible" class to menu items that have children

Reported by: digitalnature Owned by:
Priority: normal Milestone:
Component: Menus Version: 3.0.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Items from custom menus in WP 3 have classes like:

menu-item
menu-item-type-TYPE
menu-item-ID

and if the current menu item is active, also:

current-menu-item

or

current-menu-ancestor

It would useful for theme developers if you also added another class on menu items that have children, like "collapsible", or "menu-with-children" :)

this will allow designers to add things like arrows in menus easier, without having to resort to javascript hacks to check if the list element has children...

Attachments (1)

Parent-menu-item.patch​ (797 bytes ) - added by mobius5150 15 years ago.
Adds parent-menu-item class to menu items that have children

Download all attachments as: .zip

Change History (9)

#1 @nacin
16 years ago

  • Milestone Awaiting Review → Future Release

#2 @mobius5150
15 years ago

  • Cc mike@… added
  • Keywords has-patch added
  • Version 3.0.1 → 3.2

I found this while I was looking for a way to style parent menu items without using nasty javascript. I found a very nice solution ​here (basically taking the two added lines from ​this and adding it to the default WalkerNavMenu->display_element() function which simply checks to see if the menu item has children, and if so, appends the appropriate class to it.

I'll be preparing and attaching a patch ASAP.

@mobius5150
15 years ago

Adds parent-menu-item class to menu items that have children

#3 @SergeyBiryukov
15 years ago

  • Version 3.2 → 3.0.1

Version number is used to track when the bug was initially reported.

#4 @ocean90
15 years ago

Duplicate: #20191

#5 @cramdesign
15 years ago

  • Cc matt@… added

The patch looks great. Does this ticket still have any traction? Nacin's post was so long ago.

#6 @SergeyBiryukov
13 years ago

#24644 was marked as a duplicate.

#7 @celloexpressions
12 years ago

  • Resolution → duplicate
  • Status new → closed

This was fixed in #23834. There is now a class menu-item-has-children for parent menu items.

#8 @helen
12 years ago

  • Milestone Future Release
Note: See TracTickets for help on using tickets.