As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 806581 - Translate Facebook Chat
: Translate Facebook Chat
Product: Thunderbird
Classification: Client Software
Component: Instant Messaging (show other bugs)
: 18 Branch
: x86_64 Windows 7
: -- normal (vote)
: Thunderbird 19.0
Assigned To: Andreas Pettersson
Depends on:
  Show dependency treegraph
Reported: 2012-10-29 14:58 PDT by Andreas Pettersson
Modified: 2012-11-02 13:01 PDT (History)
3 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Facebook Chat label (1.22 KB, patch)
2012-10-30 10:52 PDT, Andreas Pettersson
clokep: feedback+
Details | Diff | Splinter Review
Facebook chat name (1.23 KB, patch)
2012-10-30 11:05 PDT, Andreas Pettersson
clokep: review+
Details | Diff | Splinter Review

Description User image Andreas Pettersson 2012-10-29 14:58:36 PDT
Create a new chat account. 

Facebook chat is not a proper name, it has a Swedish translation. The name is hardcoded in file
Comment 1 User image Ludovic Hirlimann [:Usul] 2012-10-30 05:00:11 PDT
Could you come up with a patch ?
Comment 2 User image Andreas Pettersson 2012-10-30 10:52:18 PDT
Created attachment 676675 [details] [diff] [review]
Facebook Chat label
Comment 3 User image Patrick Cloke [:clokep] 2012-10-30 10:56:26 PDT
Comment on attachment 676675 [details] [diff] [review]
Facebook Chat label

>diff --git a/chat/locales/en-US/ b/chat/locales/en-US/
> FacebookProtocol.prototype = {
>   __proto__: GenericProtocolPrototype,
>   get normalizedName() "facebook",
>-  get name() "Facebook Chat",
>+  get name() "network.label",
You'll need to call the translation function like on, so this would be _("network.label"). I don't find this name to be very descriptive, however. Maybe something like ""?

Would we want to do this for all of our JS protocols? (GTalk, Facebook, XMPP, IRC, Twitter) Florian?

Also, I wonder why Facebook have its own properties file, GTalk just has its strings inside of
Comment 4 User image Andreas Pettersson 2012-10-30 11:05:19 PDT
Created attachment 676677 [details] [diff] [review]
Facebook chat name
Comment 5 User image Patrick Cloke [:clokep] 2012-10-30 13:40:17 PDT
Comment on attachment 676677 [details] [diff] [review]
Facebook chat name

I'll test this out later tonight.
Comment 6 User image Patrick Cloke [:clokep] 2012-11-01 19:41:02 PDT
Comment on attachment 676677 [details] [diff] [review]
Facebook chat name

Looks good to me! Thanks. :)
Comment 7 User image Ryan VanderMeulen [:RyanVM] 2012-11-02 13:01:18 PDT

Thanks for the patch! One request - to make life easier for those checking in on your behalf, please make sure that your future patches follow the guidelines below so that all the necessary checkin metadata is present. Thanks!

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