WordPress.org

Making WordPress.org

Opened 12 months ago

Closed 2 weeks ago

Last modified 2 weeks ago

#3717 closed enhancement (fixed)

Include post date in the Reviews tab

Reported by: philclothier Owned by: ck3lee
Milestone: Priority: normal
Component: Plugin Directory Keywords: ui-feedback has-patch
Cc:

Description

Currently, the date of each review is not displayed in the main "Reviews" tab. This can be misleading for plugins which have not been reviewed/updated recently. It also seems like very useful information for adding context to older/newer versions of the plugin which might have enhancements/bugs historically.

As an example, the [WP-LESS plugin](https://wordpress.org/plugins/wp-less) has not been updated in a long time. However if you look at the [Reviews tab](https://wordpress.org/plugins/wp-less/#reviews) it appears that everything is working with the current version of WordPress. If you inspect the reviews more closely, they were actually left [several years ago](https://wordpress.org/support/topic/well-written-and-maintenance-free/) using old versions of WordPress.

Displaying a date next to the reviews adds a lot of context to the user's decision making process.

Just to confirm, I have no issues with the WP-LESS plugin. I've never even used it. The plugins was just the first one I found as a good example :)

https://i.imgur.com/WeUDow3.png

Attachments (4)

WordPress 2019-06-10 13-37-35.png (113.7 KB) - added by ck3lee 6 weeks ago.
3717.diff (2.9 KB) - added by ck3lee 6 weeks ago.
3717.1.diff (2.9 KB) - added by ck3lee 6 weeks ago.
Screen Shot 2019-07-03 at 1.42.37 pm.png (76.2 KB) - added by dd32 2 weeks ago.

Download all attachments as: .zip

Change History (23)

#1 @obenland
12 months ago

  • Keywords ui-feedback added

Hi @philclothier, welcome to Meta Trac!

Let's start with some Design feedback, @mapk do you have thoughts on this?

#2 @mapk
12 months ago

This is a great idea from @philclothier - thanks for the ticket!

While we're working on this, it might be good to add the version number of the plugin when the review was posted as well. This would be similar to the App Store.

https://cldup.com/flWEqk3dMS.png

#3 follow-up: @philclothier
12 months ago

Hi guys 👋

@mapk good thinking!

Maybe the space under the avatar could be used for any extra meta from the review? I guess there's not much space if it is a short review though.

https://i.imgur.com/im7AkEp.png

#4 in reply to: ↑ 3 @mapk
12 months ago

Replying to philclothier:

Maybe the space under the avatar could be used for any extra meta from the review? I guess there's not much space if it is a short review though.

That might be too dispersed. What about adding it to the author line like this?

https://cldup.com/Y2FceHlPdK.png

#5 @philclothier
12 months ago

Looks good to me! 👍

#6 follow-up: @obenland
12 months ago

I feel like there's a lot going on in that screenshot. Do we use pipes anywhere else in the directory?

What do you think about moving the stars up left of the heading, and then have the entire post meta be in gray?

I'm also not sure the version number makes things clearer here, my first thought was that it depicts revisions of that comment. Maybe we could skip adding that in this iteration?

#7 in reply to: ↑ 6 @mapk
12 months ago

Replying to obenland:

I feel like there's a lot going on in that screenshot. Do we use pipes anywhere else in the directory?

I'm not sure we do. We can remove those.

What do you think about moving the stars up left of the heading, and then have the entire post meta be in gray?

Here's what that might look like.

https://cldup.com/W-YR7v-hrK.png

I'm also not sure the version number makes things clearer here, my first thought was that it depicts revisions of that comment. Maybe we could skip adding that in this iteration?

I liked the version number, but am open to leaving it out for now. I'm really not sure how important it is, or whether it's useful or not.

#8 @obenland
12 months ago

You'd prefer to keep the blue link color for the author?

#9 follow-up: @tellyworth
12 months ago

The version number seems like it'd be a very useful addition if it's done right. Is there a way we can make its meaning clear?

#10 in reply to: ↑ 9 @SergeyBiryukov
12 months ago

Replying to tellyworth:

The version number seems like it'd be a very useful addition if it's done right. Is there a way we can make its meaning clear?

There was an older ticket for this: #152. It has a patch that displays the version number in the sidebar, along with the WordPress version.

The ticket was wontfixed 5 months ago. Per @Clorith, the version field would likely end up being another field users ignore or fill in wrong. WordPress version field has also been removed since then, see #3502.

Related: #1181

#11 @SergeyBiryukov
7 weeks ago

#4449 was marked as a duplicate.

#12 @SergeyBiryukov
7 weeks ago

  • Owner set to ck3lee
  • Status changed from new to assigned

@ck3lee
6 weeks ago

#13 follow-up: @ck3lee
6 weeks ago

  • Keywords has-patch added

attachment:3717.diff and screenshot are based on @mapk's latest design. Hope this works. Yeah, having the dates for each review definitely helps here.

@SergeyBiryukov, please review when you have a chance. Thank you.

#14 in reply to: ↑ 13 @dd32
6 weeks ago

Replying to ck3lee:

attachment:3717.diff and screenshot are based on @mapk's latest design. Hope this works. Yeah, having the dates for each review definitely helps here.

@SergeyBiryukov, please review when you have a chance. Thank you.

I like where this has gone, the only thing I've picked up on is that date_i18n() should be used in place of gmdate() so we've got localised months on international sites.
IIRC changing the markup here doesn't affect the markup delivered through the API to WordPress plugin info iframes, but that should be checked prior to committing this.

@ck3lee
6 weeks ago

#15 @ck3lee
6 weeks ago

Thank you for your feedback, @dd32.

Uploaded attachment:3717.1.diff with date_i18n().
Please review. I hope this works.

#16 @dd32
2 weeks ago

As it turns out we already display dates on the Reviews within the wp-admin dashboard.

I've attached a screenshot of that for reference here.

#17 @dd32
2 weeks ago

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

In 9019:

Plugin Directory: Show the review date for reviews.

Props ck3lee, mapk, philclothier.
Fixes #3717.

#18 @dd32
2 weeks ago

In 9020:

Plugin Directory: Cache bump after [9019].

See #3717.

#19 @philclothier
2 weeks ago

Thanks @dd32 and everyone involved! 🙌

Note: See TracTickets for help on using tickets.