UI: Missing comments count in admin screen

[ramiy]

See Attached image.

I'm pretty sure it's just a technical limitation, due to the fancy river of comments we have.

@ramiy: The owner field is meant either for the person that supplies the patch or that oversees the ticket (i.e. a commiter).

If you want to signal that the patch works for you, just leave a comment saying so.

Patch works for me.

Can't see a problem with the small patch.

Because we are showing the same line on the Dashboard now too (see [17797]) we should implant it in 3.2 to be consistent.

Replying to scribu:

@ramiy: The owner field is meant either for the person that supplies the patch or that oversees the ticket (i.e. a commiter).

Ok, Thanks for the feedback. I have reassignd the ticket to @solarissmoke (the patch supplier).

Didn't make it in by freeze, revisit for 3.3.

Also, we need to hide the "tabs" when the count is 0 (zero).

In posts and pages, for example, we don't see "Draft (0)" or "Trash (0)".

Same principle should be in comments, the Pending/Spam/Trash should be hidden when count in zero.

Show approved-comments before moderated-comments, based on solarissmoke patch

Show approved-comments before moderated-comments, based on solarissmoke patch

Patch tested. Looks great. Check the attached screenshot.

Punting enhancements from 3.3.

The patch doesn't work because it's showing the total number of comments, but the "All" view only displays approved and pending comments and not spammed ones.

For example with 2 approved comments, 2 pending comments and 1 spam comment I see "All (5)" but actually only 4 items are listed.

Any patch also has to deal with the JavaScript count updating too, i.e. decrementing the "All" and "Approved" counts when a comment is trashed/spammed.

Replying to duck_:

The patch doesn't work because it's showing the total number of comments, but the "All" view only displays approved and pending comments and not spammed ones.

I tested the patch few months ago, it worked for me (See the screenshot). Where there any changes that could effect the comments count?

Replying to ramiy:

I tested the patch few months ago, it worked for me (See the screenshot).

If you look at the actual number of items being displayed and the count on the right (under the search box) you should only see the two approved comments and "2 items". I think it's confusing having "All (18)" as the view 'title' when 18 comments aren't being shown.

Replying to duck_:

If you look at the actual number of items being displayed and the count on the right (under the search box) you should only see the two approved comments and "2 items". I think it's confusing having "All (18)" as the view 'title' when 18 comments aren't being shown.

It's not confusing, just like in posts/pages screen the "All" tab includes the trashed posts, in comments it includes the trash+spam comments.

I usually look at "Published" post count (don't care about drafts/trash). Same for comments, i am more intrested in "Approved" comments (don't care about spam/trash)

Replying to ramiy:

It's not confusing, just like in posts/pages screen the "All" tab includes the trashed posts, in comments it includes the trash+spam comments.

On Posts/Pages screens, the "All" tab only includes published, draft and pending posts.

Likewise, on Comments screen, the "All" tab only includes approved and pending comments.

Replying to ramiy:

It's not confusing, just like in posts/pages screen the "All" tab includes the trashed posts, in comments it includes the trash+spam comments.

Trashed posts don't count towards the "All" count on the posts screen. And with the patch applied trashed comments don't either only spammed ones.

Edit: apologies for the double posting with Sergey.

So what do you suggest to do?

You want to remove the "Spam" and "Trash" from the total count ("All" tab) ?