WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#30957 closed defect (bug) (fixed)

Twenty Fifteen: remove unused CSS class for wp_page_menu

Reported by: iamtakashi Owned by: lancewillett
Milestone: 4.2 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch commit
Focuses: Cc:

Description

While I was looking at #30722, I've realised the theme still has CSS classes for wp_page_menu which is unused in the theme - page_item_has_children, current_page_item, current_page_ancestor. We should remove them from the stylesheet.

Also I found some errors — double classes and spacing pattern.

Attachments (1)

30957.diff​ (5.3 KB) - added by iamtakashi 7 years ago.

Download all attachments as: .zip

Change History (3)

@iamtakashi
7 years ago

#1 @lancewillett
7 years ago

  • Keywords commit added

Nice cleanup, Takashi.

#2 @lancewillett
7 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In 31183:

Twenty Fifteen: remove unused CSS rules for wp_page_menu since are not used in the theme. And a bit of CSS cleanup.

Props iamtakashi, fixes #30957.

Note: See TracTickets for help on using tickets.