WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Last modified 5 years ago

#38836 reviewing enhancement

A current_page_parent CSS class is not added to relevant WP menu items if the query is for a custom post type single item

Reported by: henrywright's profile henry.wright Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Menus Keywords: has-patch has-unit-tests good-first-bug
Focuses: Cc:

Description

A current_page_parent CSS class doesn't get added to relevant post type archive menu items if the current query is for a custom post type single item.

Attachments (3)

38836.diff​ (1.0 KB) - added by henry.wright 8 years ago.
38836.2.diff​ (3.3 KB) - added by christophherr 8 years ago.
38836.3.diff​ (3.3 KB) - added by donmhico 5 years ago.
Refreshed the patch. I've re-test the patch and re-run the unit tests as well. Everything looks good.

Download all attachments as: .zip

Change History (15)

#1 @henry.wright
8 years ago

Steps to reproduce

  1. Add a post type archive item to a WP menu
  2. Visit a single item of that post type

Note that the current_page_parent CSS class isn't added to the menu item.

#2 @henry.wright
8 years ago

  • Component changed from General to Menus
  • Type changed from defect (bug) to enhancement

@henry.wright
8 years ago

#3 @henry.wright
8 years ago

  • Keywords has-patch added

Please find 38836.diff attached.

#4 @welcher
8 years ago

  • Keywords needs-unit-tests added

Patch looks ok to me. Let's get some unit test in place for it.

@christophherr
8 years ago

#5 @christophherr
8 years ago

  • Keywords has-unit-tests added; needs-unit-tests removed

38836.2.diff adds unit tests.

#6 @desrosj
6 years ago

  • Keywords needs-refresh good-first-bug added
  • Milestone changed from Awaiting Review to Future Release
  • Version 4.6.1 deleted

The latest patch needs a refresh to apply cleanly.

#7 @diouldebalde
5 years ago 

<?php
[[Image([''''])]]

@donmhico
5 years ago

Refreshed the patch. I've re-test the patch and re-run the unit tests as well. Everything looks good.

#8 @donmhico
5 years ago

  • Keywords needs-refresh removed

#9 @SergeyBiryukov
5 years ago

  • Milestone changed from Future Release to 5.3
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

This ticket was mentioned in ​Slack in #core by sergey. ​View the logs.
5 years ago

#11 @davidbaumwald
5 years ago

@SergeyBiryukov The patch looks good and clean. Is this something we can include for 5.3 Beta 1 today? If not, feel free to update the milestone to whatever you think is realistic.

#12 @davidbaumwald
5 years ago

  • Milestone changed from 5.3 to Future Release

With version 5.3 Beta 1 releasing shortly, the deadline for enhancements is now passed. This is being moved to Future Release.

Note: See TracTickets for help on using tickets.