fix: disable general notification button and add install recommendati…

…on (#107)
MaSch0212 · Jun 26, 2024 · e0afe6d · e0afe6d
1 parent fbe80cd
commit e0afe6d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/client/src/app/components/user-settings/user-settings.component.html b/src/client/src/app/components/user-settings/user-settings.component.html
@@ -53,7 +53,9 @@ <h1 class="m-0">{{ translations.nav_settings() }}</h1>
   } @else {
     @if (settings(); as settings) {
       <p-card [header]="translations.settings_notifications_title()">
-        <div class="flex flex-col gap-4">
+        <div class="flex flex-row gap-4">
+          <!-- this toggle caused confusion during alpha testing -->
+          <!--
           <div class="flex flex-row items-center gap-4">
             <div class="grow">
               {{ translations.settings_notifications_enabledOnAllDevices() }}
@@ -70,7 +72,7 @@ <h1 class="m-0">{{ translations.nav_settings() }}</h1>
                 [showTrigger]="settingsSaveStates.enabledOnAllDevices | async"
               />
             </div>
-          </div>
+          </div> -->
 
           <div class="flex flex-col gap-1">
             <div class="flex flex-row items-center gap-4">

diff --git a/src/client/src/app/i18n/de.json b/src/client/src/app/i18n/de.json
@@ -42,7 +42,7 @@
       "enabledOnAllDevices": "Auf allen Geräten aktiviert",
       "eanbled": "Auf diesem Gerät aktiviert",
       "errors": {
-        "notSupported": "Benachrichtigungen werden von diesem Browser nicht unterstützt.",
+        "notSupported": "Benachrichtigungen werden von diesem Browser nicht unterstützt. Gegenebenfalls muss die Webseite auf dem Home-Bildschirm installiert werden.",
         "notGranted": {
           "title": "Erlaubnis über Benachrichtigungen nicht erteilt",
           "description": "Um Benachrichtigungen erhalten zu können, musst du dies in den Systemeinstellungen aktivieren, oder die App erneut installieren."

diff --git a/src/client/src/app/i18n/en.json b/src/client/src/app/i18n/en.json
@@ -42,7 +42,7 @@
       "enabledOnAllDevices": "Enabled on all devices",
       "eanbled": "Enabled on this device",
       "errors": {
-        "notSupported": "Notifications are not supported by this browser.",
+        "notSupported": "Notifications are not supported by this browser. Probably this website needs to be installed onto the homescreen.",
         "notGranted": {
           "title": "Notification permission not granted",
           "description": "To receive notifications, you must activate this in the system settings or reinstall the app."