From a3e76d1c9b303160a2241b8b0f1862c88a2f48f2 Mon Sep 17 00:00:00 2001
From: coffiarts <github@coffiarts.com>
Date: Sat, 16 Dec 2023 15:59:04 +0100
Subject: [PATCH] Get rid of permanent messages

---
 src/sheet-locker/lang/de.json      | 12 ++----------
 src/sheet-locker/lang/en.json      |  4 ++--
 src/sheet-locker/scripts/config.js |  6 ------
 src/sheet-locker/scripts/main.js   | 20 ++++----------------
 4 files changed, 8 insertions(+), 34 deletions(-)

diff --git a/src/sheet-locker/lang/de.json b/src/sheet-locker/lang/de.json
index 4aab4de..2957f8d 100644
--- a/src/sheet-locker/lang/de.json
+++ b/src/sheet-locker/lang/de.json
@@ -25,14 +25,6 @@
         "name": "UI-Meldungen beim Umschalten",
         "hint": "Zeige kurze Bannermeldungen an, wenn das Panning & Zooming aus- oder eingeschaltet wird (funktioniert auch beim Umschalten via Macro)."
       },
-      "notifyPermanentlyWhileLOCKED": {
-        "name": "Status 'Gesperrt' permanent anzeigen",
-        "hint": "Im Sperrzustand permanente Bannermeldung anzeigen (bis zur Entsperrung). Nur wirksam, wenn \"UI-Meldungen beim Umschalten\" aktiviert ist."
-      },
-      "notifyPermanentlyWhileUNLOCKED": {
-        "name": "Status 'Entsperrt' permanent anzeigen",
-        "hint": "Im entsperrten Zustand permanente Bannermeldung anzeigen (bis zur Sperrung). Nur wirksam, wenn \"UI-Meldungen beim Umschalten\" aktiviert ist."
-      },
       "alertGMOnReject": {
         "name": "Spielleiter alarmieren",
         "hint": "Bildschirmwarnung für den SL anzeigen, wenn ein Spieler ein gesperrten Charakterbogen ändern will."
@@ -43,11 +35,11 @@
       },
       "overlayIconLocked": {
         "name": "Status Icon bei Sperrung (optional)",
-        "hint": "Welches Icon soll auf Tokens und Akteuren angezeigt werden, wenn sie gesperrt sind (leer lassen, falls kein Icon angezeigt werden soll)?"
+        "hint": "Welches Icon soll auf Tokens und Akteuren angezeigt werden, wenn sie gesperrt sind (leer lassen, falls kein Icon angezeigt werden soll)? (Standard: modules/sheet-locker/artwork/lock-red-closed.png)"
       },
       "overlayIconOpen": {
         "name": "Status Icon bei Entsperrung (optional)",
-        "hint": "Welches Icon soll auf Tokens und Akteuren angezeigt werden, wenn sie freigeschaltet sind (leer lassen, falls kein Icon angezeigt werden soll)?"
+        "hint": "Welches Icon soll auf Tokens und Akteuren angezeigt werden, wenn sie freigeschaltet sind (leer lassen, falls kein Icon angezeigt werden soll)? (Standard: modules/sheet-locker/artwork/lock-green-open.png)"
       }
     },
     "chatInfoContent": {
diff --git a/src/sheet-locker/lang/en.json b/src/sheet-locker/lang/en.json
index 77389d9..5399a27 100644
--- a/src/sheet-locker/lang/en.json
+++ b/src/sheet-locker/lang/en.json
@@ -43,11 +43,11 @@
       },
       "overlayIconLocked": {
         "name": "Status Icon when locked (optional)",
-        "hint": "Pick which icon to display on tokens and actors when locked (leave empty to disable icon)?"
+        "hint": "Pick which icon to display on tokens and actors when locked (leave empty to disable icon)? (Default: modules/sheet-locker/artwork/lock-red-closed.png)"
       },
       "overlayIconOpen": {
         "name": "Status Icon when unlocked (optional)",
-        "hint": "Pick which icon to display on tokens and actors when unlocked (leave empty to disable icon)?"
+        "hint": "Pick which icon to display on tokens and actors when unlocked (leave empty to disable icon)? (Default: modules/sheet-locker/artwork/lock-green-open.png)"
       }
     },
     "chatInfoContent": {
diff --git a/src/sheet-locker/scripts/config.js b/src/sheet-locker/scripts/config.js
index f1a8bf5..e4d21ec 100644
--- a/src/sheet-locker/scripts/config.js
+++ b/src/sheet-locker/scripts/config.js
@@ -39,12 +39,6 @@ export class Config {
             notifyOnChange: {
                 scope: 'world', config: true,  type: Boolean, default: true
             },
-            notifyPermanentlyWhileLOCKED: {
-                scope: 'world', config: true, type: Boolean, default: false
-            },
-            notifyPermanentlyWhileUNLOCKED: {
-                scope: 'world', config: true, type: Boolean, default: false
-            },
             lockForGM: {
                 scope: 'world', config: true, type: Boolean, default: false
             },
diff --git a/src/sheet-locker/scripts/main.js b/src/sheet-locker/scripts/main.js
index c0912d7..1f0d479 100644
--- a/src/sheet-locker/scripts/main.js
+++ b/src/sheet-locker/scripts/main.js
@@ -267,31 +267,19 @@ function stateChangeUIMessage() {
 
     if (Config.setting('notifyOnChange')) {
 
-        let isPermanent = (
-            SheetLocker.isActive &&  Config.setting('notifyPermanentlyWhileLOCKED')
-            ||
-            !SheetLocker.isActive &&  Config.setting('notifyPermanentlyWhileUNLOCKED')
-        );
-
-        // Clear previous permanent msg (if any)
-        if (permanentUIMsgID != null) {
-            ui.notifications.remove(permanentUIMsgID);
-        }
-
         if (SheetLocker.isActive) {
-            permanentUIMsgID = ui.notifications.error(`[${Config.data.modTitle}] ${message}`, {
-                permanent: isPermanent,
+            ui.notifications.error(`[${Config.data.modTitle}] ${message}`, {
+                permanent: false,
                 localize: false,
                 console: false
             });
         } else {
-            permanentUIMsgID = ui.notifications.info(`[${Config.data.modTitle}] ${message}`, {
-                permanent: isPermanent,
+            ui.notifications.info(`[${Config.data.modTitle}] ${message}`, {
+                permanent: false,
                 localize: false,
                 console: false
             });
         }
-        if (!isPermanent) permanentUIMsgID = null;
     }
     Logger.info(message);
 }