Comment on attachment 9102584
1575512-glue.patch

Review of attachment 9102584:
-----------------------------------------------------------------

It's a pity we have to add these hacks for the current localization 
infrastructure. But it is maybe also a bit unusual we cache strings in the 
backend like this.
I like this non-simplified version better. It could be useful to have that 
gInitializeStringsDone variable indicate if we think the strings are OK, 
instead of blindly assuming they are (in the simplified version).
Thanks for pursuing this with the various approaches you have tried.

::: mail/components/nsIMailGlue.idl
@@ +17,5 @@
>     *
>     */
>    void sanitize(in nsIDOMWindow aParentWindow);
>  
> +  void finalUiStartupDone(out boolean aDone);

Can this be made 'attribute boolean finalUiStartupDone' ?

::: mailnews/base/util/nsMsgDBFolder.cpp
@@ +2727,2 @@
>    nsresult rv;
> +  if (!aForce) {

Can you add some TODO comment that all these hacks can go if m-c changes
the localization infrastructure so that the localized strings are
available immediately?

@@ +3182,5 @@
>           (name.LowerCaseEqualsLiteral("outbox") && nonEnglishApp());
>  }
>  
>  NS_IMETHODIMP nsMsgDBFolder::SetPrettyName(const nsAString &name) {
> +  initializeStrings();

Could we actually check the return value of initializeStrings? It can
fail at many sports and reports the errors, but we never check it at the
callers.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to thunderbird in Ubuntu.
https://bugs.launchpad.net/bugs/1847772

Title:
  E-mail folder names are not localized in thunderbird 68

Status in Mozilla Thunderbird:
  In Progress
Status in thunderbird package in Ubuntu:
  Confirmed

Bug description:
  After updating to Thunderbird 68.1.1 from the Ubuntu archive, folder
  names Inbox, Drafts, Sent, Archives, Spam, Trash all appear in English
  in the French UI. The rest of the UI is in French.

  This issue is still present in Thunderbird 1:68.1.2+build1-0ubuntu1.

  This problem doesn’t exist in Thunderbird 68.1.1 nor in 68.1.2
  downloaded directly from Mozilla.

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/1847772/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to