properly localize plurals of number of matches in Addressbook search

RESOLVED FIXED in Thunderbird 53.0

Status

defect
--
trivial
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: aceman, Assigned: aceman)

Tracking

Trunk
Thunderbird 53.0
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird52 fixed, thunderbird53 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

11.16 KB, patch
iann_bugzilla
: review+
Details | Diff | Splinter Review
https://dxr.mozilla.org/comm-central/rev/6b8c7cd17855f7de5eccb0ea9efcd5dbccc68573/mail/locales/en-US/chrome/messenger/addressbook/addressBook.properties#111 hardcodes 2 plural forms of the string "X matches found". This is fine for English but not other languages.

Fix this by using the usual PluralForm.jsm.
Posted patch patch (obsolete) — Splinter Review
Attachment #8805867 - Flags: review?(mkmelin+mozilla)
Attachment #8805867 - Flags: review?(iann_bugzilla)
Comment on attachment 8805867 [details] [diff] [review]
patch

Review of attachment 8805867 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=mkmelin

::: mail/base/content/ABSearchDialog.js
@@ +31,5 @@
>      UpdateCardView();
>    },
> +  onCountChanged: function(aTotal) {
> +    let statusText;
> +    if (aTotal == 0)

nit: braces for this too
Attachment #8805867 - Flags: review?(mkmelin+mozilla) → review+
Posted patch patch v1.1Splinter Review
Thanks.
Attachment #8805867 - Attachment is obsolete: true
Attachment #8805867 - Flags: review?(iann_bugzilla)
Attachment #8807667 - Flags: review?(iann_bugzilla)
Comment on attachment 8807667 [details] [diff] [review]
patch v1.1

LGTM r/a=me
Attachment #8807667 - Flags: review?(iann_bugzilla) → review+
Thanks.
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/b8ec04edc0e3e7248d01607653cb0bffb132aa6f
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 53.0
Dashboard shows an error for this string, even though it has been translated correctly:

https://l10n.mozilla.org/dashboard/compare?run=650844

https://hg.mozilla.org/l10n-central/pl/rev/0cf430ee6773
Is this problem still happening? Is a newer run of the tool showing it?
Please make a new bug for this and we can try some experiments.
New bug is #1320570
Depends on: 1320570
Comment on attachment 8807667 [details] [diff] [review]
patch v1.1

I will uplift this since otherwise bug 1319409 doesn't apply and I don't feel like messing around with a special patch here.
Attachment #8807667 - Flags: approval-comm-aurora?
Comment on attachment 8807667 [details] [diff] [review]
patch v1.1

Aurora (TB 52):
https://hg.mozilla.org/releases/comm-aurora/rev/5082b3870a86205345bb8a2d23456602dad7e7b7
Attachment #8807667 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.