Show a special banner when a langpack is installed

stasm · Jan 8, 2015 · d42d9ef · d42d9ef
1 parent 43a6766
commit d42d9ef
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/apps/system/js/app_install_manager.js b/apps/system/js/app_install_manager.js
@@ -313,9 +313,10 @@ var AppInstallManager = {
     var manifest = app.manifest || app.updateManifest;
     var appManifest = new ManifestHelper(manifest);
     var name = appManifest.name;
-    var _ = navigator.mozL10n.get;
-    var msg = _('app-install-success', { appName: name });
-    this.systemBanner.show(msg);
+    var l10nId = appManifest.role === 'langpack' ?
+      'langpack-install-success' : 'app-install-success';
+    this.systemBanner.show(
+      navigator.mozL10n.get(l10nId, { appName: name }));
   },
 
   checkSetupQueue: function ai_checkSetupQueue() {

diff --git a/apps/system/locales/system.en-US.properties b/apps/system/locales/system.en-US.properties
@@ -180,6 +180,7 @@ app-download-stop-button=Stop Download
 not-enough-space=Not enough space
 not-enough-space-message=There is not enough space to install this app. Free up space by deleting old apps or media, and try installing again from the original source.
 app-install-success={{ appName }} installed
+langpack-install-success={{ appName }} installed. Go to Settings > Languages to enable it.
 
 #setup dialog
 setup=Setup