WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 22 months ago

Last modified 18 months ago

#18615 closed defect (bug) (fixed)

Quick Editing a child page makes it appear to not be a child any more

Reported by: SergeyBiryukov Owned by: markjaquith
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.1
Component: Quick/Bulk Edit Keywords: has-patch 3.5-early
Focuses: Cc:

Description

[15500] brought back #8444:

To reproduce: Go to the Pages screen and Quick Edit any child page. Upon saving, the page will appear to be all the way on the left, not indented like all the other child pages.

This is a mere visual/JavaScript error, the page is still a child properly, as reloading the Pages screen will show.

Attachments (2)

18615.patch (1.8 KB) - added by SergeyBiryukov 3 years ago.
18615.2.patch (3.2 KB) - added by ssamture 2 years ago.
Adding post(page) level check syntax.(include #19474)

Download all attachments as: .zip

Change History (14)

SergeyBiryukov3 years ago

comment:1 Ipstenu3 years ago

  • Cc ipstenu@… added

comment:3 SergeyBiryukov2 years ago

Same for hierarchical taxonomies: #19132

ssamture2 years ago

Adding post(page) level check syntax.(include #19474)

comment:6 SergeyBiryukov2 years ago

  • Milestone changed from Awaiting Review to 3.4

comment:7 nacin2 years ago

  • Keywords 3.5-early added
  • Milestone changed from 3.4 to Future Release

Let's fix this in conjunction with #19474 for 3.5.

comment:8 SergeyBiryukov22 months ago

  • Milestone changed from Future Release to 3.5

comment:9 markjaquith22 months ago

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

In [21192]:

Prevent child pages from being visually promoted to the top level after Quick Edit. props ssamture. fixes #18615

comment:10 follow-up: markjaquith22 months ago

SergeyBiryukov, I'm not completely sure what your patch is trying to do...

comment:11 in reply to: ↑ 10 SergeyBiryukov22 months ago

Replying to markjaquith:

SergeyBiryukov, I'm not completely sure what your patch is trying to do...

Never mind, it was an earlier attempt. ssamture's patch is proper.

comment:12 SergeyBiryukov18 months ago

#22142 was marked as a duplicate.

Note: See TracTickets for help on using tickets.