The "Confirm and Restart" button on the Preference tab for switching to another locale doesn't work
Categories
(Thunderbird :: Preferences, defect)
Tracking
(thunderbird68+ fixed, thunderbird69 fixed)
People
(Reporter: nONoNonO, Assigned: jorgk-bmo)
References
(Regression)
Details
(Keywords: regression)
When you have an extra language pack installed and exposed the GUI by toggling the pref intl.multilingual.enabled to true, the button to change to new locale doesn't work. See the following message in the errorconsole:
TypeError: event is undefined
confirmLanguageChange chrome://messenger/content/preferences/advanced.js:728
showConfirmLanguageChangeMessageBar chrome://messenger/content/preferences/advanced.js:705
When looking at the code, I see that in addEventListener no event is passed to the function:
https://searchfox.org/comm-central/source/mail/components/preferences/advanced.js#746
button.addEventListener("command", () => gAdvancedPane.confirmLanguageChange());
Maybe something like this fixes it?
button.addEventListener("command", (event) => gAdvancedPane.confirmLanguageChange(event));
Assignee | ||
Comment 1•5 years ago
|
||
I didn't say "Assign to Richard", I said, NI him. That stuff is ported from M-C and it looks different there:
https://searchfox.org/mozilla-central/rev/c606cdd6d014fee4034da1702d484c0d41b604c9/browser/components/preferences/in-content/main.js#810
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
I'm pushing a fix now, sadly I can't test it on trunk, but it won't get any worse ;-) - Richard, can you please look whether it's still working on trunk.
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/08f782cdbba0
Revert part of rev 423c3668781b (bug 1547581) to fix new language 'confirm and restart' button. a=backout DONTBUILD
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 4•5 years ago
|
||
TB 68 beta 1:
https://hg.mozilla.org/releases/comm-beta/rev/cfc0d18189f4af2f99d2bd6bba902bb3049d9c63
Should be fixed, we'll have to see.
Assignee | ||
Comment 5•5 years ago
|
||
(In reply to Jorg K (GMT+2) from comment #2)
I'm pushing a fix now, sadly I can't test it on trunk, but it won't get any worse ;-) - Richard, can you please look whether it's still working on trunk.
Axel mentioned on IRC that it's broken and he'll file a bug for it. Apparently some Fluent "no bundle was loaded" error.
Thanks, the change may work but reverting this goes against the idea of bug 1547581.
Can we rather add the event argument as in comment 0?
Assignee | ||
Comment 7•5 years ago
|
||
Well, that code needs to be kept 100% in sync with M-C for easier porting. If they don't do it, we don't do it. You can apply your new goodness anywhere else ;-)
Assignee | ||
Comment 8•5 years ago
|
||
Hmm, I can't test this since I'm running into bug 1558859. How did you manage to install the additional message in the first place? Is it fixed in beta 1 build 2?
Comment 9•5 years ago
|
||
A installed language should be shown already in the menulist on Advanced itself. And here you can select your language.
I tried it on tip and this works, it asks for restart. I see bug 1558859 too. This dialog is to sort/add/remove the languages.
Reporter | ||
Comment 10•5 years ago
|
||
I downloaded 68.0b1 from http://archive.mozilla.org/pub/thunderbird/nightly/latest-comm-beta/ and the language pack from http://archive.mozilla.org/pub/thunderbird/nightly/latest-comm-beta-l10n/win32/xpi/ and installed it from the add-ons manager.
Assignee | ||
Comment 11•5 years ago
|
||
Sure, but to set a new language you run into bug 1558859. Anyway, I'll put that on the beta and then we'll see.
EDIT: Sorry, comment #9, the languages are already on the menu list. Case closed. I verified the beta patch on a try build.
Updated•4 years ago
|
Description
•