Closed Bug 1710825 Opened 3 months ago Closed 2 months ago

Improve wording of public bulk mail notification string (style/logic)

Categories

(Thunderbird :: General, task)

Tracking

(thunderbird_esr78 unaffected, thunderbird89 wontfix)

RESOLVED FIXED
90 Branch
Tracking Status
thunderbird_esr78 --- unaffected
thunderbird89 --- wontfix

People

(Reporter: thomas8, Assigned: thomas8)

Details

Attachments

(3 files, 1 obsolete file)

STR

  1. compose with 15 recipients in To and CC to trigger public bulk mail notification
  2. look at the wording of the first sentence of the notification

Actual

The 15 recipients in To and Cc can see the address of each other. You can avoid disclosing recipients by using Bcc instead.

There's something wrong with the language style and logic in the current bulk mail notification:

  • "of" is typically used with things, so it's odd to use it with people.
  • using "address" (instead of "addresses") feels logically wrong and unnecessarily playing down the real-world effect this will have. Even if there are only two people, they will see each other's addresses (not address), more so with 3+ people. Grammar of good authority corroborates this:

    Lisa and Kate hated each other’s husbands. (Lisa hated Kate’s husband and Kate hated Lisa’s husband.)
    https://dictionary.cambridge.org/grammar/british-grammar/each-other-one-another

  • "can" (present tense) is also not quite correct, as at the present time of composing, recipients cannot see each other's addresses yet. If the message gets sent, they will...

Expected

Let's improve the wording like this:

The 15 recipients in To and Cc will see each other's addresses. You can avoid disclosing recipients by using Bcc instead.

We also want this for translations, so I've updated the l10n-id to be in sync with related IDs.

This changes the string per comment 0.

many-public-recipients-info = The {$count} recipients in To and Cc will see each other's addresses. You can avoid disclosing recipients by using Bcc instead.

We also want this for translations, so I've updated the l10n-id to be in sync with related IDs (many-public-recipients-bcc, many-public-recipients-ignore).

Attachment #9221558 - Flags: review?(lasana)
Comment on attachment 9221558 [details] [diff] [review]
1710825_publicBulkRecipientsNotificationString.diff

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

Sure!
Attachment #9221558 - Flags: review?(lasana) → review+

Thanks Lasana!

Comment on attachment 9221558 [details] [diff] [review]
1710825_publicBulkRecipientsNotificationString.diff

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

It is actually singular address, I think. Using plural makes it sound like all (other) potential addresses would be visible.
Singular form "each other's address" has 4 times more hits on google as well.

Please note: Review procedure is that a peer or owner should be involved in the review flow.
Attachment #9221558 - Flags: review-

Alright, no problem! I've changed it back to singular which is certainly good:
The {$count} recipients in To and Cc will see each other's address. ...

(In reply to Magnus Melin [:mkmelin] from comment #5)

Review of attachment 9221558 [details] [diff] [review] ⧉[details] [diff] [review]:
It is actually singular address, I think. Using plural makes it sound like
all (other) potential addresses would be visible.

Yeah, I hear you. Sort of... maybe. Seems that it works both ways (singular or plural), and plural is getting more common and also considered correct [1][2].

Singular form "each other's address" has 4 times more hits on google as well.

True, but at the same time, "each other's faces" (with quotes) has more than twice as many hits on google over "each other's face", so apparently using the plural does not imply that everyone has more than one face...

[1] https://jakubmarian.com/each-others-vs-each-others-in-english/
[2] https://dictionary.cambridge.org/grammar/british-grammar/each-other-one-another

Attachment #9222064 - Flags: review?(mkmelin+mozilla)
Attachment #9222064 - Flags: review?(mkmelin+mozilla) → review+
Target Milestone: --- → 90 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/fc57dfa5d1b6
Improve wording of public bulk mail notification string. r=lasana,mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/d32912fff6e3
followup to fix fluent apostrophs. rs=fluent-lint DONTBUILD

(In reply to Pulsebot from comment #8)

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/d32912fff6e3
followup to fix fluent apostrophs. rs=fluent-lint DONTBUILD

Thanks!

Attachment #9221558 - Attachment is obsolete: true

Fwiw, TB 78 is probably wontfix, but not unaffected as it does have the original typo (inconsequential):
https://hg.mozilla.org/releases/comm-release/file/9921e1929e744e2f5e64163f036c215f2960fc1f/mail/components/compose/content/MsgComposeCommands.js#l4137
function udpateAddressingInputAriaLabel(row) {

Sorry, my comment 10 ended up on the wrong bug, was meant for bug 1711270.

You need to log in before you can comment on or make changes to this bug.