1951142 - Angry emoji named as Yell emoji

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Camaleon]

Attachment: TB_ANGRY_EMOJI_NAMED_AS_YELL_EMOJI

The English string for the Angry emoji 😠 (Unicode U+1F620) is (wrongly?) named as Yell emoji 😱 (Unicode U+1F631).

I guess this also affects the whole of localizations for that emoji.
Eg., In Spanish (es-ES) the Angry emoji is translated as «Gritar» (😠 Gritar), while should be «Enfadado» (😠 Enfadado).
(See attached image)

(from Pontoon¹)
CONTEXT smiley10Cmd.label•mail/chrome/messenger/messengercompose/messengercompose.dtd•Thunderbird
CREATED April 10, 2017 11:48:16 AM

¹https://pontoon.mozilla.org/es-ES/thunderbird/mail/chrome/messenger/messengercompose/messengercompose.dtd/?string=86617

Comment 1

[Wayne Mery (:wsmwk)]

(assuming this affects 128, but might not be explicitly uplifted once fixed)

Comment 2

[Camaleon]

(In reply to Wayne Mery (:wsmwk) from comment #1)

(assuming this affects 128, but might not be explicitly uplifted once fixed)

Yes, sorry, I missed adding the affected version of TB.
I can reproduce the bug in 128.8.1esr (64-bit), running Debian 11 (bullseye).
But being a localization issue, I guess this is widely spread in any TB release (also nightly/beta versions).

Comment 3

[Magnus Melin [:mkmelin]]

Does seem it was always wrong. We're using &#128544 - https://www.compart.com/en/unicode/U+1F620

Comment 4

[Camaleon]

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

Does seem it was always wrong. We're using &#128544 - https://www.compart.com/en/unicode/U+1F620

Mmmm... Just to clarify.
It's not the emoji icon that fails BUT the English string that names it.
The icon is right (angry face) but not the Yell string describing it.

Comment 5

[Corey Bryant]

Attachment: Bug 1951142 - Rename Yell emoji to Angry emoji. r=#thunderbird-reviewers

The emoji was incorrectly named Yell and should be Angry.

Comment 6

[Corey Bryant]

This looked like a good opportunity for me to see the whole lifecycle of l10n, so I've submitted a patch for this change.

Comment 7

[Magnus Melin [:mkmelin]]

Corey: ready to land?

Comment 8

[Corey Bryant]

Oops, yes

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/2361387db217
Rename Yell emoji to Angry emoji. r=mkmelin