Opened 10 years ago
Last modified 5 years ago
#29299 reviewing defect (bug)
Plural forms not available for some strings
Reported by: | pavelevap | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | |
Component: | Options, Meta APIs | Keywords: | has-patch 2nd-opinion settings-api |
Focuses: | administration, ui-copy | Cc: |
Description
There are several strings which do not allow specific plurals in Settings - Reading.
<?php _e( 'posts' ); ?>
<?php _e( 'items' ); ?>
It looks strange when somebody inserts for example number 4 and our plural form is translated only for number 5.
Best solution would be probably some kind of string "Number of posts on page" (without ending "posts" string). But I am not sure about it...
See: https://core.trac.wordpress.org/browser/trunk/src/wp-admin/options-reading.php#L126
Attachments (2)
Change History (22)
#3
@
10 years ago
This is closely related to #13673 and #15576.
Per nacin's comment:
We wouldn't customize this based on the number the user has entered. It should be whatever the maximum plural form is, if I had to guess.
I agree that it looks weird though, perhaps we could reconsider this.
#4
@
10 years ago
Yes, but this could be easier...
What about "Number of posts to show on page" and remove "posts" string entirely?
And "Number of recent posts to show in RSS" and remove "items" string entirely?
#5
@
9 years ago
- Component changed from I18N to Text Changes
- Focuses administration added
- Keywords needs-patch added
#6
@
8 years ago
- Keywords has-patch added; needs-patch removed
Patch added which amends the strings.
#7
follow-up:
↓ 8
@
8 years ago
- Milestone changed from Awaiting Review to 4.7
- Owner set to SergeyBiryukov
- Status changed from new to reviewing
#8
in reply to:
↑ 7
@
8 years ago
Replying to SergeyBiryukov:
Seems consistent with [31696], though I'd like to see if we can address comment:1 as well.
Most of the items in Discussions settings are grouped, and have multiple controls i.e. a checkbox and an input field or dropdown.
They are going to need more thought, and they are mutually exclusive to those in Reading settings. A post-commit discussion or a new ticket may be best IMHO.
This ticket was mentioned in Slack in #core by stevenkword. View the logs.
8 years ago
#10
@
8 years ago
29299.2.diff is what I had in mind:
- Doesn't change the strings, but makes them support proper plural forms.
- Handles the strings on Discussion Settings screen as well.
- Adds translator comments.
#12
@
8 years ago
I would vote for changing strings, because adding plural forms (from latest patch) will not work immediatelly when somebody change values (before saving them).
#13
@
8 years ago
- Keywords commit removed
The singular and plural forms in 29299.2.diff aren't an improvement because the form won't always be correct when selecting different values.
Does this also affect the various inline fields on the Discussion Settings screen?