WordPress.org

Make WordPress Core

Opened 4 years ago

Last modified 10 months ago

#14687 new feature request

Add "collapsible" class to menu items that have children

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

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 3 years ago.
Adds parent-menu-item class to menu items that have children

Download all attachments as: .zip

Change History (7)

comment:1 nacin3 years ago

  • Milestone changed from Awaiting Review to Future Release

comment:2 mobius51503 years ago

  • Cc mike@… added
  • Keywords has-patch added
  • Version changed from 3.0.1 to 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.

mobius51503 years ago

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

comment:3 SergeyBiryukov3 years ago

  • Version changed from 3.2 to 3.0.1

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

comment:4 ocean902 years ago

Duplicate: #20191

comment:5 cramdesign2 years ago

  • Cc matt@… added

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

comment:6 SergeyBiryukov10 months ago

#24644 was marked as a duplicate.

Note: See TracTickets for help on using tickets.