[sr] Strings should have 3 forms in plural rule #7

RESOLVED FIXED

Status

Mozilla Localizations
sr / Serbian
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Takeshi Nishimura, Assigned: Filip Miletic)

Tracking

({fixed1.9.0.9})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
The following strings should have 3 forms according to plural rule #7.
Correct ex. |downloadsTitleFiles|
http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/sr/file/tip/toolkit/chrome/mozapps/downloads/downloads.properties#l110

|newAddonsNotificationMsg2|
http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/sr/file/tip/toolkit/chrome/mozapps/extensions/extensions.properties#l100

|activeDownloads|, |pausedDownloads|
http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/sr/file/tip/browser/chrome/browser/browser.properties#l129

|editBookmark.removeBookmarks.label|
http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/sr/file/tip/browser/chrome/browser/browser.properties#l141
(Assignee)

Updated

9 years ago
Assignee: nobody → filmil
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
(Reporter)

Updated

9 years ago
Blocks: 451586
(Assignee)

Comment 1

9 years ago
Created attachment 364646 [details] [diff] [review]
The plurals patch for Serbian
Attachment #364646 - Flags: review?(l10n)
Attachment #364646 - Flags: approval1.9.0.8?
Attachment #364646 - Flags: approval1.9.0.7?
(Assignee)

Comment 2

9 years ago
(In reply to comment #0)
> The following strings should have 3 forms according to plural rule #7.
> Correct ex. |downloadsTitleFiles|
> http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/sr/file/tip/toolkit/chrome/mozapps/downloads/downloads.properties#l110

This one already has 3 plurals.

> |newAddonsNotificationMsg2|
> http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/sr/file/tip/toolkit/chrome/mozapps/extensions/extensions.properties#l100

This and the others are corrected by the attached patch.  Please review.
Attachment #364646 - Flags: approval1.9.0.7?

Updated

9 years ago
Attachment #364646 - Flags: review?(l10n)
Attachment #364646 - Flags: review+
Attachment #364646 - Flags: approval1.9.0.8?
Attachment #364646 - Flags: approval1.9.0.8+

Comment 3

9 years ago
Comment on attachment 364646 [details] [diff] [review]
The plurals patch for Serbian

r/a=me. Please land this with a check-in comment referencing this bug, describing the change and mentioning the approval. Something like "bug 478660, Strings should have 3 forms in plural rule #7, r/a=l10n@mozilla.com" would work.

Please use the fixed1.9.0.8 and verified1.9.0.8 keywords to track landing and testing.

Comment 4

9 years ago
Filip, do you have time to land this on CVS?
(Assignee)

Comment 5

9 years ago
Yes, but I have problems committing them now.  Please see m.d.l10n.
(Assignee)

Comment 6

9 years ago
Hm... committed...
(Assignee)

Updated

9 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Keywords: fixed1.9.0.8
Resolution: --- → FIXED
Keywords: fixed1.9.0.8 → fixed1.9.0.9
You need to log in before you can comment on or make changes to this bug.