From 372d74db69a8f99b78c83eb07c0cdf3339ce3b44 Mon Sep 17 00:00:00 2001
From: JeDaYoshi <hi@jeda.im>
Date: Sun, 4 Jul 2021 20:22:49 +0000
Subject: [PATCH] Add warning for HTTPS requirement on notifications

---
 client/components/Windows/Settings.vue | 9 +++++++++
 client/js/store.js                     | 2 ++
 2 files changed, 11 insertions(+)

diff --git a/client/components/Windows/Settings.vue b/client/components/Windows/Settings.vue
index 898e1613..1c6e264f 100644
--- a/client/components/Windows/Settings.vue
+++ b/client/components/Windows/Settings.vue
@@ -306,6 +306,7 @@
 					<input
 						id="desktopNotifications"
 						:checked="$store.state.settings.desktopNotifications"
+						:disabled="$store.state.desktopNotificationState === 'nohttps'"
 						type="checkbox"
 						name="desktopNotifications"
 					/>
@@ -316,6 +317,14 @@
 					>
 						<strong>Warning</strong>: Notifications are not supported by your browser.
 					</div>
+					<div
+						v-if="$store.state.desktopNotificationState === 'nohttps'"
+						id="warnBlockedDesktopNotifications"
+						class="error"
+					>
+						<strong>Warning</strong>: Notifications are only supported over HTTPS
+						connections.
+					</div>
 					<div
 						v-if="$store.state.desktopNotificationState === 'blocked'"
 						id="warnBlockedDesktopNotifications"
diff --git a/client/js/store.js b/client/js/store.js
index 4c32ce15..83ce7f86 100644
--- a/client/js/store.js
+++ b/client/js/store.js
@@ -12,6 +12,8 @@ function detectDesktopNotificationState() {
 		return "unsupported";
 	} else if (Notification.permission === "granted") {
 		return "granted";
+	} else if (!window.isSecureContext) {
+		return "nohttps";
 	}
 
 	return "blocked";