WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#9495 closed defect (bug) (invalid)

current_page_item class too liberal

Reported by: Denis-de-Bernardy Owned by:
Milestone: Priority: normal
Severity: minor Version: 2.8
Component: Template Keywords: needs-patch 2nd-opinion
Focuses: Cc:

Description

The Page walker sets this in the li tag.

Is it me, or not all browsers support css such as the following?

.current_page_item > a {
  font-weight: bold;
}

Shouldn't we give the class to the anchor tag directly?

Change History (2)

comment:1 @Viper007Bond6 years ago

Erm?

.current_page_item a {
  font-weight: bold;
}

comment:2 @Viper007Bond6 years ago

  • Milestone 2.8 deleted
  • Resolution set to invalid
  • Status changed from new to closed

Giving the CSS class to the li tag provides a ton more flexibility and it'd break a bajillion themes who expect the current output.

Note: See TracTickets for help on using tickets.