Make WordPress Core

Opened 20 years ago

Closed 20 years ago

#3253 closed defect (bug) (fixed)

Give parnent of current page/category a CSS class

Reported by: mdawaffe Owned by: markjaquith
Priority: high Milestone: 2.1
Component: Template Version: 2.1
Severity: normal Keywords:
Cc: Focuses:

Description

Attached gives current_page_parent and current-cat-parent to the parent of the current object in wp_list_pages() and wp_list_categories().

Attachments (1)

3253.diff​ (1.3 KB ) - added by mdawaffe 20 years ago.

Download all attachments as: .zip

Change History (7)

@mdawaffe
20 years ago

#1 @markjaquith
20 years ago

Recommend that dashes be usedin class names and IDs where ever possible: current-page-parent

​http://developer.mozilla.org/en/docs/Underscores_in_class_and_ID_Names

#2 @mdawaffe
20 years ago

I agree, but we're already using current_page; I went for current_page_parent for the sake of parallelism. It'd be very confusing to people if one were one way and the other were the other.

How about we give the current page both current_page and current-page and give the parent just current-page-parent?

#3 @markjaquith
20 years ago

That'll break some plugins, unfortunately, that are filtering wp_list_pages. Of course, it wouldn't be the first time that the CSS classes for pages was changed... I think it changed in 2.0

#4 @markjaquith
20 years ago

  • Owner changed from anonymous to markjaquith
  • Status new → assigned

Should probably pass by reference for get_page() and get_category

Let's stick with the underscore... I don't like them being different and changing them to dashes would break countless themes and plugins.

#5 @matt
20 years ago

  • Priority normal → high

#6 @ryan
20 years ago

  • Resolution → fixed
  • Status assigned → closed

(In [4576]) Give CSS class to parent of current page/cat. Props mdawaffe. fixes #3253

Note: See TracTickets for help on using tickets.