Make WordPress Core

Opened 7 years ago

Closed 13 months ago

Last modified 13 months ago

#38834 closed defect (bug) (fixed)

Quick Edit for future dated posts shows 'published' instead of 'scheduled'

Reported by: ipstenu's profile Ipstenu Owned by: joedolson's profile joedolson
Milestone: 6.3 Priority: normal
Severity: normal Version:
Component: Quick/Bulk Edit Keywords: has-patch commit
Focuses: ui, accessibility, javascript Cc:

Description

If you make a post and schedule it for the future, quick edit properly shows it's dropdown as 'scheduled' if you want to change post status.

If, however, you have a future dated post that you *want* to schedule, but it's in draft, the option is paradoxically 'Published.'

This leads to uncertainty as to what the next step actually is. The dropdown should alter itself based on what your current saved date is (it not ajaxifiedly adapt on the fly).

Attachments (4)

Screen Shot 2016-11-16 at 9.41.28 AM.png (47.9 KB) - added by Ipstenu 7 years ago.
Example: That should say schedule not publish
38834.diff (949 bytes) - added by MrFlannagan 7 years ago.
The patch checks if the current date and time is before that of a post then instead of removing the "Schedule" option on an inline post edit in draft mode, it leaves "Schedule" and removes "Publish"
38834.1.diff (992 bytes) - added by sabernhardt 3 years ago.
patch refreshed
2023-03-11_15-25-58.png (42.2 KB) - added by oglekler 14 months ago.

Download all attachments as: .zip

Change History (20)

@Ipstenu
7 years ago

Example: That should say schedule not publish

@MrFlannagan
7 years ago

The patch checks if the current date and time is before that of a post then instead of removing the "Schedule" option on an inline post edit in draft mode, it leaves "Schedule" and removes "Publish"

This ticket was mentioned in Slack in #core by jjj. View the logs.


5 years ago

#5 @noisysocks
3 years ago

  • Keywords needs-refresh added

38834.diff no longer applies cleanly.

@sabernhardt
3 years ago

patch refreshed

#6 @sabernhardt
3 years ago

  • Component changed from Editor to Quick/Bulk Edit
  • Focuses javascript added
  • Keywords has-patch needs-testing added; needs-refresh removed
  • Milestone changed from Awaiting Review to Future Release

38834.1.diff refreshes the first patch and replaces the deprecated $.now() method.

#7 @sabernhardt
3 years ago

Related/duplicate: #30761 ...oops, already mentioned in comment:1

Last edited 3 years ago by sabernhardt (previous) (diff)

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


16 months ago

#9 @joedolson
16 months ago

  • Focuses accessibility added
  • Owner set to joedolson
  • Status changed from new to accepted

Adding accessibility focus to this since the duplicate #30761 has it, and I'm closing that in favor of this ticket.

#10 @joedolson
16 months ago

#30761 was marked as a duplicate.

#11 @joedolson
15 months ago

  • Milestone changed from Future Release to 6.3

This ticket was mentioned in Slack in #core-test by oglekler. View the logs.


14 months ago

This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.


14 months ago

#14 @oglekler
14 months ago

Patch 38834.1 looks fine. It's tested with WordPress trunk (6.3-alpha-55505-src), Google Chrome 110.0.5481.105 and Fire Fox 110.0.1 browsers.

This ticket was mentioned in PR #4248 on WordPress/wordpress-develop by @joedolson.


13 months ago
#15

Adds var declaration on status

Trac ticket: https://core.trac.wordpress.org/ticket/38834

#16 @joedolson
13 months ago

  • Keywords commit added; needs-testing removed

#17 @joedolson
13 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 55560:

Quick/Bulk Edit: Show scheduled in status for future drafts.

Change the quick edit status select to use the 'Scheduled' status instead of the 'Published' status or posts where the date is in the future.

Props ipstenu, MrFlannagan, sabernhardt, oglekler.
Fixes #38834.

@joedolson commented on PR #4248:


13 months ago
#18

Fixed in r55560

Note: See TracTickets for help on using tickets.