[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-commits] [tor-browser] 22/57: Bug 41369: Improve Firefox language settings for multi-lingual packages
This is an automated email from the git hooks/post-receive script.
richard pushed a commit to branch tor-browser-102.5.0esr-12.0-2
in repository tor-browser.
commit f0e75b468e979ea4aa2733755d58a7300947908e
Author: Pier Angelo Vendrame <pierov@xxxxxxxxxxxxxx>
AuthorDate: Tue Oct 18 19:02:18 2022 +0200
Bug 41369: Improve Firefox language settings for multi-lingual packages
Change the language selector to be sorted by language code, rather than
name, and to display the language code to the user.
Bug 41372: Handle Japanese as a special case in preferences on macOS
Japanese is treated in a special way on macOS. However, seeing the
Japanese language tag could be confusing for users, and moreover the
language name is not localized correctly like other langs.
Bug 41378: Tell users that they can change their language at the first start
With multi-lingual builds, Tor Browser matches the user's system
language, but some users might want to change it.
So, we tell them that it is possible, but only once.
---
browser/base/content/browser.xhtml | 2 +
browser/base/content/languageNotification.js | 67 ++++++++++++++++++++++
browser/base/jar.mn | 2 +
browser/components/preferences/main.inc.xhtml | 2 +-
browser/components/preferences/main.js | 24 +++++++-
.../locales/en-US/browser/languageNotification.ftl | 10 ++++
6 files changed, 104 insertions(+), 3 deletions(-)
diff --git a/browser/base/content/browser.xhtml b/browser/base/content/browser.xhtml
index 32fca7fdc89c..0750982c3483 100644
--- a/browser/base/content/browser.xhtml
+++ b/browser/base/content/browser.xhtml
@@ -85,6 +85,7 @@
#ifdef NIGHTLY_BUILD
<link rel="localization" href="preview/firefoxView.ftl"/>
#endif
+ <link rel="localization" href="browser/languageNotification.ftl"/>
<title data-l10n-id="browser-main-window-title"></title>
@@ -112,6 +113,7 @@
Services.scriptloader.loadSubScript("chrome://browser/content/places/places-menupopup.js", this);
Services.scriptloader.loadSubScript("chrome://browser/content/search/autocomplete-popup.js", this);
Services.scriptloader.loadSubScript("chrome://browser/content/search/searchbar.js", this);
+ Services.scriptloader.loadSubScript("chrome://browser/content/languageNotification.js", this);
window.onload = gBrowserInit.onLoad.bind(gBrowserInit);
window.onunload = gBrowserInit.onUnload.bind(gBrowserInit);
diff --git a/browser/base/content/languageNotification.js b/browser/base/content/languageNotification.js
new file mode 100644
index 000000000000..73a5da0b58aa
--- /dev/null
+++ b/browser/base/content/languageNotification.js
@@ -0,0 +1,67 @@
+"use strict";
+
+// Show a prompt to suggest to the user that they can change the UI language.
+// Show it only the first time, and then do not show it anymore
+window.addEventListener("load", async () => {
+ const PREF_NAME = "intl.language_notification.shown";
+
+ if (Services.prefs.getBoolPref(PREF_NAME, false)) {
+ return;
+ }
+
+ // Already customized, we do not suggest to change it again...
+ if (Services.prefs.getCharPref("intl.locale.requested", "") !== "") {
+ // ... and we never show the notification, either
+ Services.prefs.setBoolPref(PREF_NAME, true);
+ return;
+ }
+
+ // In sync with our changes on browser/components/preferences/main.js for
+ // tor-browser#41369 and tor-browser#41372.
+ const code =
+ Services.locale.appLocaleAsBCP47 === "ja-JP-macos"
+ ? "ja"
+ : Services.locale.appLocaleAsBCP47;
+ const language = Services.intl
+ .getLocaleDisplayNames(undefined, [code], { preferNative: true })[0]
+ .replace(/\s*\(.+\)$/g, "");
+
+ // We want to determine whether the current locale was chosen based on the
+ // system locales, in which case langauge negotiation returns a match, or
+ // whether it simply defaulted to en-US.
+ const matchingSystem = !!Services.locale.negotiateLanguages(
+ // Since intl.locale.requested is empty, we expect requestedLocales to match
+ // the user's system locales.
+ Services.locale.requestedLocales,
+ Services.locale.availableLocales
+ ).length;
+ const label = await document.l10n.formatValue(
+ matchingSystem
+ ? "language-notification-label-system"
+ : "language-notification-label",
+ { language }
+ );
+
+ const buttons = [
+ {
+ "l10n-id": "language-notification-button",
+ callback() {
+ openPreferences("general-language");
+ },
+ },
+ ];
+
+ gNotificationBox.appendNotification(
+ "language-notification",
+ {
+ label,
+ priority: gNotificationBox.PRIORITY_INFO_HIGH,
+ },
+ buttons
+ );
+
+ // We do not wait for the user to either click on the button or dismiss the
+ // notification: after we have shown it once, we take for granted that the
+ // user has seen it and we never show it again.
+ Services.prefs.setBoolPref(PREF_NAME, true);
+});
diff --git a/browser/base/jar.mn b/browser/base/jar.mn
index d2b075a50f39..c57663d64408 100644
--- a/browser/base/jar.mn
+++ b/browser/base/jar.mn
@@ -106,6 +106,8 @@ browser.jar:
content/browser/spotlight.js (content/spotlight.js)
* content/browser/default-bookmarks.html (content/default-bookmarks.html)
+ content/browser/languageNotification.js (content/languageNotification.js)
+
% override chrome://global/content/netError.xhtml chrome://browser/content/certerror/aboutNetError.xhtml
# L10n resources and overrides.
diff --git a/browser/components/preferences/main.inc.xhtml b/browser/components/preferences/main.inc.xhtml
index 0deb010bbefd..d655ac37aed1 100644
--- a/browser/components/preferences/main.inc.xhtml
+++ b/browser/components/preferences/main.inc.xhtml
@@ -322,7 +322,7 @@
</groupbox>
<!-- Languages -->
-<groupbox id="languagesGroup" data-category="paneGeneral" hidden="true">
+<groupbox id="languagesGroup" data-category="paneGeneral" hidden="true" data-subcategory="language">
<label><html:h2 data-l10n-id="language-header"/></label>
<vbox id="browserLanguagesBox" align="start" hidden="true">
diff --git a/browser/components/preferences/main.js b/browser/components/preferences/main.js
index f0864a5cfff4..171198b181a7 100644
--- a/browser/components/preferences/main.js
+++ b/browser/components/preferences/main.js
@@ -1027,8 +1027,28 @@ var gMainPane = {
available,
{ preferNative: true }
);
- let locales = available.map((code, i) => ({ code, name: localeNames[i] }));
- locales.sort((a, b) => a.name > b.name);
+ let locales = available.map((code, i) => {
+ let name = localeNames[i].replace(/\s*\(.+\)$/g, "");
+ if (code === "ja-JP-macos") {
+ // Mozilla codebases handle Japanese in macOS in different ways,
+ // sometimes they call it ja-JP-mac and sometimes they call it
+ // ja-JP-macos. The former is translated to Japanese when specifying
+ // preferNative to true, the latter is not. Since seeing ja-JP-macos
+ // would be confusing anyway, we treat it as a special case.
+ // See tor-browser#41372 and Bug 1726586.
+ name =
+ Services.intl.getLocaleDisplayNames(undefined, ["ja"], {
+ preferNative: true,
+ })[0] + " (ja)";
+ } else {
+ name += ` (${code})`;
+ }
+ return {
+ code,
+ name,
+ };
+ });
+ locales.sort((a, b) => a.code.localeCompare(b.code));
let fragment = document.createDocumentFragment();
for (let { code, name } of locales) {
diff --git a/browser/locales/en-US/browser/languageNotification.ftl b/browser/locales/en-US/browser/languageNotification.ftl
new file mode 100644
index 000000000000..2ad06bf5c2d5
--- /dev/null
+++ b/browser/locales/en-US/browser/languageNotification.ftl
@@ -0,0 +1,10 @@
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+
+# $language is the language Tor Browser is displayed in (already translated)
+language-notification-label-system = { -brand-short-name } has set your display language to { $language } based on your system’s language.
+# This is shown when the system language is not supported, so we fall back to another language instead.
+# $language is the language Tor Browser is displayed in (already translated).
+language-notification-label = { -brand-short-name } has set your display language to { $language }.
+language-notification-button = Change Language…
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits