Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#6027 closed defect (bug) (fixed)

Human diff in comment listing uses the post time, instead of comments

Reported by: nbachiyski's profile nbachiyski Owned by:
Milestone: 2.5 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

Steps to reproduce:

  • Post a comment
  • Go to Comments in the admin menu, less than a day after the comment
  • Note the From <postname>, <some time ago> part

Expected: the <some time ago> part is the difference between the current time and the comment time

Actual: the <some time ago> part is the difference between the current time and the post time

Attachments (1)

human-diff-comments.diff (1.1 KB) - added by nbachiyski 17 years ago.

Download all attachments as: .zip

Change History (5)

#1 @ryan
17 years ago

The mockups look like the Date column should hold the human time for the comment and that the "From..." should indeed be the human time for the post. We're not doing human time in the Date column. The From... time seems to be right.

#2 @nbachiyski
17 years ago

If this is the idea, the text can say something like "From <post-name>, post written <time>". Otherwise it is misleading.

IMHO, having the exact comment time is a lot more useful than the post time.

#3 @nbachiyski
17 years ago

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

Fixed in [7231].

#4 @ryan
17 years ago

I still think it looks wrong to have comment date next to the post title, especially when the date of the comment is in another column. I like looking at the post date to see how long ago the post was made, a good indication of whether the comment is likely to be spam.

Note: See TracTickets for help on using tickets.