Make WordPress Core

Opened 13 years ago

Closed 9 years ago

Last modified 9 years ago

#18641 closed enhancement (fixed)

Date and post status are the wrong way around in post lists

Reported by: johnbillion's profile johnbillion Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.4 Priority: normal
Severity: normal Version: 3.3
Component: Text Changes Keywords: has-patch has-screenshots
Focuses: ui, administration Cc:

Description

On a post listing screen, a draft post that has been modified within the last 24 hours will show the following in the 'Date' column:

X hours ago
Last Modified

This doesn't make a lot of sense. Better would be:

Last Modified
X hours ago

Ideally the table cell should be treated as a whole to aid l10n and to improve the English presentation.

Attachments (5)

18641.patch (1.4 KB) - added by johnbillion 13 years ago.
18641.diff (1.3 KB) - added by wonderboymusic 9 years ago.
Screen Shot 2015-09-11 at 10.18.01 AM.png (90.3 KB) - added by ryan 9 years ago.
Before, Macnchrome, 4.3
IMG_4714.PNG (195.6 KB) - added by ryan 9 years ago.
After, iPhone 6+
Screen Shot 2015-09-11 at 10.22.14 AM.png (103.4 KB) - added by ryan 9 years ago.
After, Macnchrome

Download all attachments as: .zip

Change History (14)

@johnbillion
13 years ago

#1 @johnbillion
13 years ago

  • Keywords has-patch added

Patch which swaps the order of the date and post status and uses the entire string for l10n.

#2 @iseulde
11 years ago

  • Component changed from Administration to Text Changes

#3 @chriscct7
9 years ago

  • Keywords needs-refresh added
  • Milestone changed from Awaiting Review to 4.4
  • Owner set to chriscct7
  • Status changed from new to assigned

#4 @chriscct7
9 years ago

  • Focuses ui administration added

#5 @wonderboymusic
9 years ago

  • Owner changed from chriscct7 to wonderboymusic

@wonderboymusic
9 years ago

#6 @wonderboymusic
9 years ago

  • Keywords needs-refresh removed

18641.diff simplifies this - just move the strings before the time

#7 @wonderboymusic
9 years ago

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

In 34005:

In WP_Posts_List_Table::column_date(), the order of status and time should be switched. It doesn't make sense for phrases like "Last Modified" to occur AFTER the date. Should be before.

Props johnbillion, wonderboymusic.
Fixes #18641.

#8 @ryan
9 years ago

  • Keywords needs-screenshots added

I'm going to add some before and after screenshots. I'm attempting before and after shots of all visual changes on multiple devices. Screenshots are used in visual changelogs and the Today in the Nightly posts, all of which increases awareness of our usability and make ui review easier. https://make.wordpress.org/core/tag/today-in-the-nightly/

If you'd like to help collect visuals, we want before and after screenshots on at least two devices, with one of them preferably being a phone. We're increasing our mobile/touch awareness by testing and capturing all visual changes from phones. Provide whatever screenshots you can. The Flow Patrol team will handle the remainder. Developers, taking screenshots of your patches and commits really helps ui/ux review and flow patrol. https://make.wordpress.org/flow/

@ryan
9 years ago

Before, Macnchrome, 4.3

@ryan
9 years ago

After, iPhone 6+

@ryan
9 years ago

After, Macnchrome

#9 @ryan
9 years ago

  • Keywords has-screenshots added; needs-screenshots removed
Note: See TracTickets for help on using tickets.