i18 problems for Comment section

[pavelevap]

1) Function wp_comment_trashnotice() uses for example string "Comment by %s moved to the trash". Placeholder is for user, but in our translation this placeholder has to be in the end to make it sense, that means "... autorem byl %s." But there is problem with additonal space character which is inserted automatically by Javascript, so the string is "... autorem byl %s ." I tried to fix it, but not sure if I could broke anything else, so please look at attached patch.

2) I do not see the need to use list on help tab for Comments, it is also attached in this patch. I browsed related admin pages and there are lists only when there is colon in the first sentence. I am not sure, maybe it should be reformulated a bit...

3) There is following sentence in the help tab: "The small bubble with the number in it shows how many comments that post has received." But as I can see, number shows not all received comments, but only approved comments. I am not sure if Help text or number itself is wrong...

4) And the last one - there are some strings in Help and Comments section which refer to "Post", but comments can be added to Attachments, Pages, etc. Especially, link "View Post" is not very good, because it is highligted many times on this admin page, even if there are only attachments or books...

[nacin]

Point 1: Valid.

Point 2: Agreed.

Point 3: The bubble shows the number of approved comments, but is a highlight color if there are pending comments. The help text needs a small tweak.

Point 4: We have the view_item label, which defaults to "View Post" or "View Page". It should have been used here.

[nacin]

Check that, view_item *is* used there. While we should probably add a view_item for the attachment post type, I'm not sure what it should be. We call it "Attachment Post" ("Attachment Post URL") when inserting media, and "Attachment Page" when inserting a gallery. "View Attachment" seems okay in this context, though, as it's not like we're differentiating between the image itself and the page it is on.

[nacin]

Marking for review from Jane.

[nacin]

In [19804]:

Don't add in an extra space when inserting a name into the string produced by wp_comment_trashnotice(). props pavelevap, see #19788.

[nacin]

In [19806]:

Convert a bulleted list to paragraphs in the Moderating Comments help tab. props pavelevap, see #19788.

[SergeyBiryukov]

Traced the extra space in point 1 to [12139].

[nacin]

Replying to SergeyBiryukov:

Traced the extra space in point 1 to [12139].

Likewise, but could not ascertain why it was added. When the helper was added earlier than that, see ​http://core.trac.wordpress.org/browser/trunk/wp-admin/includes/template.php?rev=12008#L2362, it was not ideal for i18n but it would have put spaces on each side.

[nacin]

For review from Jane:

• The help text on the Comments screen says "The small bubble with the number in it shows how many comments that post has received." This is not accurate — the bubble shows the number of approved comments only. It is highlighted blue if there are pending comments. What should the new help text say?

• On the Comments screen, it says "View Post" or "View Page" in the right column. For comments on attachments, should it say "View Attachment"? "View Attachment Page"? "View Attachment Post"?

[lessbloat]

Replying to nacin:

For review from Jane:

I can't speak for Jane, but here are my thoughts:

• The help text on the Comments screen says "The small bubble with the number in it shows how many comments that post has received." This is not accurate — the bubble shows the number of approved comments only. It is highlighted blue if there are pending comments. What should the new help text say?

Changed it to read, "The small bubble with the number in it shows the number of approved comments that post has received." in 19788.2.diff

• On the Comments screen, it says "View Post" or "View Page" in the right column. For comments on attachments, should it say "View Attachment"? "View Attachment Page"? "View Attachment Post"?

This appears to already be set in trunk as "View Attachment Page" (line 59 of wp-includes/post.php), which sounds good to me.

[ryan]

In [22275]:

Improve the contextual help for the comment bubble in the 'In Response To' column. Props lessbloat. fixes #19788