From f8dce875f233321f3208a541544f420c079fce21 Mon Sep 17 00:00:00 2001
From: "J. Ryan Stinnett" <jryans@gmail.com>
Date: Tue, 26 Mar 2019 10:09:04 +0000
Subject: [PATCH] Clarify devices affected by notification settings

This clarifies that the notification settings only apply to the current device.
This also tries to apply the spirit of
https://github.com/matrix-org/matrix-react-sdk/pull/1995 (authored by @aidalgol)
which wanted to remove "web" from the label, since there's also a desktop
client.
---
 src/components/views/settings/Notifications.js | 4 ++--
 src/i18n/strings/en_EN.json                    | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/views/settings/Notifications.js b/src/components/views/settings/Notifications.js
index b8f8279bb0..f344f2c897 100644
--- a/src/components/views/settings/Notifications.js
+++ b/src/components/views/settings/Notifications.js
@@ -837,7 +837,7 @@ module.exports = React.createClass({
 
                     <LabelledToggleSwitch value={SettingsStore.getValue("notificationsEnabled")}
                                           onChange={this.onEnableDesktopNotificationsChange}
-                                          label={_t('Enable desktop notifications')} />
+                                          label={_t('Enable desktop notifications for this device')} />
 
                     <LabelledToggleSwitch value={SettingsStore.getValue("notificationBodyEnabled")}
                                           onChange={this.onEnableDesktopNotificationBodyChange}
@@ -845,7 +845,7 @@ module.exports = React.createClass({
 
                     <LabelledToggleSwitch value={SettingsStore.getValue("audioNotificationsEnabled")}
                                           onChange={this.onEnableAudioNotificationsChange}
-                                          label={_t('Enable audible notifications in web client')} />
+                                          label={_t('Enable audible notifications for this device')} />
 
                     { emailNotificationsRows }
 
diff --git a/src/i18n/strings/en_EN.json b/src/i18n/strings/en_EN.json
index ef4bc75d27..1f4d97e408 100644
--- a/src/i18n/strings/en_EN.json
+++ b/src/i18n/strings/en_EN.json
@@ -497,9 +497,9 @@
     "Advanced notification settings": "Advanced notification settings",
     "There are advanced notifications which are not shown here": "There are advanced notifications which are not shown here",
     "You might have configured them in a client other than Riot. You cannot tune them in Riot but they still apply": "You might have configured them in a client other than Riot. You cannot tune them in Riot but they still apply",
-    "Enable desktop notifications": "Enable desktop notifications",
+    "Enable desktop notifications for this device": "Enable desktop notifications for this device",
     "Show message in desktop notification": "Show message in desktop notification",
-    "Enable audible notifications in web client": "Enable audible notifications in web client",
+    "Enable audible notifications for this device": "Enable audible notifications for this device",
     "Off": "Off",
     "On": "On",
     "Noisy": "Noisy",