Making WordPress.org

Opened 3 months ago

Closed 5 days ago

Last modified 5 days ago

#7761 closed enhancement (fixed)

Add submitted date in Plugin Controls box

Reported by: rabmalin's profile rabmalin Owned by: tellyworth's profile tellyworth
Milestone: Priority: normal
Component: Plugin Directory Keywords: has-patch
Cc:

Description

In the Plugin Controls box, we have different values like status, version, updated date, etc. It would be great if we could add there submitted date also. I believe it will be helpful while plugin is being reviewed.

Attachments (4)

Screenshot 2024-08-30 at 12.27.12 PM.png (14.7 KB) - added by rabmalin 3 months ago.
Screenshot 2024-09-25 at 4.23.48 PM.png (53.0 KB) - added by tellyworth 3 months ago.
Output as tested on sandbox.
Screenshot 2024-09-26 at 4.34.50 PM.png (48.0 KB) - added by tellyworth 3 months ago.
7761-submit-date.2.diff (722 bytes) - added by tellyworth 3 months ago.
Edited patch

Download all attachments as: .zip

Change History (14)

#1 @davidperez
3 months ago

Ok, I think it's helpful too.

This ticket was mentioned in PR #369 on WordPress/wordpress.org by @rabmalin.


3 months ago
#2

  • Keywords has-patch added

#3 @tellyworth
3 months ago

  • Owner set to tellyworth
  • Status changed from new to accepted

#4 @tellyworth
3 months ago

I just tested this. Had to remove the strtotime() call as noted in GH. The output is fine but could be a little neater; see screenshot.

If you want to tweak the output a bit I can wait on that, otherwise I'm happy to commit it with just the strtotime change (I can do that).

@tellyworth
3 months ago

Output as tested on sandbox.

#5 @rabmalin
3 months ago

We could omit time portion if it keeps output in single line.

#6 @tellyworth
3 months ago

It does on my screen. I also took the liberty of using Y-m-d instead of Y/m/d. Perhaps using the locale standard date would be ideal but I think that's universally unambiguous.

@tellyworth
3 months ago

Edited patch

#7 @rabmalin
3 months ago

It looks great for me.

#8 @dd32
5 days ago

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

In 14244:

Plugin Directory: Reviewers: Include the Submitted date in the controls widget.

Props rabmalin, tellyworth, dd32.
Fixes #7761.

#9 @dd32
5 days ago

I dropped the (Y-m-d) because it was causing awkward wrapping on my screen.

I replaced the title of the span with the Y-m-d where as the patches before had it as the unix epoch. That is to say, you should be able to hover on the X months ago for both the Last Updated & Submitted dates to see the actual timestamps if you need it.

If you need a copyable date, LMK and we can probably make it work with a on-hover / etc.. I did look at popover tooltips but it's not quite there (IMHO).

#10 @dd32
5 days ago

In 14246:

Pugin Directory: Guess the submission date for pre-2017-April plugins, avoids showing it as '55 years ago'.

Followup to [14244].
See #7761.

Note: See TracTickets for help on using tickets.