989389 - [B2G][Email] Confirmation messages when setting or un-setting mutliple email flags are grammatically incorrect

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[Joshua Mitchell (Inactive)]

Attachment: logcat.txt

Description:
When setting multiple (2 as an example) email flags you are presented with a message "2 messages flag set"(screenshot 1) or "2 messages flag removed"(screenshot2).
The grammatically correct versions of these messages would have the plural s on the flags resulting in "2 message flags set" and "2 message flags removed"

Repro Steps: Pre-condition: Email account set up with multiple messages in inbox
1) Update a buri to BuildID: 20140326000201
2) Access email app
3) Open the account's Inbox and enter Edit Message View
4) Select more than 1 email
5) Select the flag icon
6) Observe the confirmation message

Actual:
"2 messages flag set"

Expected:
"2 message flags set"

1.4 Environmental Variables:
Device: buri 1.4 MOZ
BuildID: 20140326000201
Gaia: 7e705dd4718d528974d99ac31866318d7e201152
Gecko: 4889124accfa
Version: 30.0a2
Firmware Version: v1.2-device.cfg

Notes:

Repro frequency: 100%
See attached: logcat and 2 screenshots

Comment 1

[Joshua Mitchell (Inactive)]

This does repro on buri 1.3

1.3 Environmental Variables:
Device: buri 1.3 MOZ
BuildID: 20140326004002
Gaia: 812838ad0fabf51fa14435af562ddac6d26fa936
Gecko: ba97efb0da4b
Version: 28.0
Firmware Version: v1.2-device.cfg

Flag confirmation messages are grammatically incorrect

Comment 2

[Joshua Mitchell (Inactive)]

Attachment: Screenshot1.png

Comment 3

[Joshua Mitchell (Inactive)]

Attachment: Screenshot2.png

Comment 4

[robert.sajdok]

Attachment: Proposed fix.

Comment 5

[James Burke [:jrburke]]

Comment on attachment 8400629 [details] [review]
Proposed fix.

Asking :stas for some l10n guidance, as he seemed to touch some of these strings before. I could see a case for using "Flag set on N messages" as an alternate, but also not sure yet if that will fit. This pull request change to use "N message flags set" also seems to work too, and will likely just take this change unless :stas indicates a preference otherwise.

Comment 6

[James Burke [:jrburke]]

Comment on attachment 8400629 [details] [review]
Proposed fix.

Going to go ahead and merge this as it is better than what we have now, and it is easy to revisit later if need be.

Comment 7

[James Burke [:jrburke]]

Merged to gaia master:
https://github.com/mozilla-b2g/gaia/commit/3fa3556397fd8c371e2d5ab3337ce222bb0bcfca

from pull request:
https://github.com/mozilla-b2g/gaia/pull/17901