Ticket #19788 (accepted defect (bug))
i18 problems for Comment section
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.4 |
| Component: | I18N | Version: | 3.3.1 |
| Severity: | normal | Keywords: | ux-feedback |
| Cc: | pavelevap@… |
Description
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...
Attachments
Change History
- Owner set to nacin
- Status changed from new to accepted
- Milestone changed from Awaiting Review to 3.4
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.
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.
comment:7
follow-up:
↓ 8
SergeyBiryukov — 3 weeks ago
Traced the extra space in point 1 to [12139].
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.

