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