From e6ad6a552ef26197cc87cd42ebe8c1ba2f2f1ece Mon Sep 17 00:00:00 2001
From: Paul Makles <paulmakles@gmail.com>
Date: Sun, 18 Sep 2022 12:05:48 +0100
Subject: [PATCH 1/2] feat: add focus mode

---
 external/components                       |  2 +-
 external/lang                             |  2 +-
 external/revolt.js                        |  2 +-
 src/components/common/user/UserIcon.tsx   |  2 ++
 src/components/common/user/UserStatus.tsx |  4 ++++
 src/context/Theme.tsx                     |  8 +++++---
 src/lib/ContextMenus.tsx                  |  9 +++++++++
 src/mobx/stores/NotificationOptions.ts    |  9 ++++++++-
 src/styles/_context-menu.scss             |  4 ++++
 yarn.lock                                 | 10 +++++-----
 10 files changed, 40 insertions(+), 12 deletions(-)

diff --git a/external/components b/external/components
index 55709411..ab0fc1af 160000
--- a/external/components
+++ b/external/components
@@ -1 +1 @@
-Subproject commit 557094115d17a363eb6fb8f27c6e697e049ce623
+Subproject commit ab0fc1af739f877c8f416dcd7e47d25aacbb7eea
diff --git a/external/lang b/external/lang
index 14d51d68..843cece2 160000
--- a/external/lang
+++ b/external/lang
@@ -1 +1 @@
-Subproject commit 14d51d68a579ee3796837cd16deda4281f83c923
+Subproject commit 843cece2854185651e17765bcb78109d22eba769
diff --git a/external/revolt.js b/external/revolt.js
index 044b1b91..ab064f41 160000
--- a/external/revolt.js
+++ b/external/revolt.js
@@ -1 +1 @@
-Subproject commit 044b1b91ee583bfb89fff140ecda29a32f6535aa
+Subproject commit ab064f41a4154acee7c1751ea5b59d9ee4345a0b
diff --git a/src/components/common/user/UserIcon.tsx b/src/components/common/user/UserIcon.tsx
index 3c96b9a6..fca6cde3 100644
--- a/src/components/common/user/UserIcon.tsx
+++ b/src/components/common/user/UserIcon.tsx
@@ -25,6 +25,8 @@ export function useStatusColour(user?: User) {
     return user?.online && user?.status?.presence !== "Invisible"
         ? user?.status?.presence === "Idle"
             ? theme.getVariable("status-away")
+            : user?.status?.presence === "Focus"
+            ? theme.getVariable("status-focus")
             : user?.status?.presence === "Busy"
             ? theme.getVariable("status-busy")
             : theme.getVariable("status-online")
diff --git a/src/components/common/user/UserStatus.tsx b/src/components/common/user/UserStatus.tsx
index 36e866c4..137cf630 100644
--- a/src/components/common/user/UserStatus.tsx
+++ b/src/components/common/user/UserStatus.tsx
@@ -32,6 +32,10 @@ export default observer(({ user, tooltip }: Props) => {
             return <Text id="app.status.idle" />;
         }
 
+        if (user.status?.presence === "Focus") {
+            return <Text id="app.status.focus" />;
+        }
+
         if (user.status?.presence === "Invisible") {
             return <Text id="app.status.offline" />;
         }
diff --git a/src/context/Theme.tsx b/src/context/Theme.tsx
index 9d8ec18b..f079c9e9 100644
--- a/src/context/Theme.tsx
+++ b/src/context/Theme.tsx
@@ -31,6 +31,7 @@ export type Variables =
     | "tooltip"
     | "status-online"
     | "status-away"
+    | "status-focus"
     | "status-busy"
     | "status-streaming"
     | "status-invisible";
@@ -283,6 +284,7 @@ export const PRESETS: Record<string, Theme> = {
         "tertiary-foreground": "#3a3a3a",
         "status-online": "#3ABF7E",
         "status-away": "#F39F00",
+        "status-focus": "#4799F0",
         "status-busy": "#F84848",
         "status-streaming": "#977EFF",
         "status-invisible": "#A5A5A5",
@@ -310,6 +312,7 @@ export const PRESETS: Record<string, Theme> = {
         "tertiary-foreground": "#848484",
         "status-online": "#3ABF7E",
         "status-away": "#F39F00",
+        "status-focus": "#4799F0",
         "status-busy": "#F84848",
         "status-streaming": "#977EFF",
         "status-invisible": "#A5A5A5",
@@ -336,9 +339,8 @@ export const generateVariables = (theme: Theme) => {
         if (colour) {
             const [r, g, b] = colour;
             return `--${key}: ${theme[key]}; --${key}-rgb: ${r}, ${g}, ${b};`;
-        } 
-            return `--${key}: ${theme[key]};`;
-        
+        }
+        return `--${key}: ${theme[key]};`;
     });
 };
 
diff --git a/src/lib/ContextMenus.tsx b/src/lib/ContextMenus.tsx
index 6aeb7c41..3298a4aa 100644
--- a/src/lib/ContextMenus.tsx
+++ b/src/lib/ContextMenus.tsx
@@ -1110,6 +1110,15 @@ export default function ContextMenus() {
                                 <div className="indicator idle" />
                                 <Text id={`app.status.idle`} />
                             </MenuItem>
+                            <MenuItem
+                                data={{
+                                    action: "set_presence",
+                                    presence: "Focus",
+                                }}
+                                disabled={!isOnline}>
+                                <div className="indicator focus" />
+                                <Text id={`app.status.focus`} />
+                            </MenuItem>
                             <MenuItem
                                 data={{
                                     action: "set_presence",
diff --git a/src/mobx/stores/NotificationOptions.ts b/src/mobx/stores/NotificationOptions.ts
index 91a5fe35..d0b79c94 100644
--- a/src/mobx/stores/NotificationOptions.ts
+++ b/src/mobx/stores/NotificationOptions.ts
@@ -157,6 +157,8 @@ export default class NotificationOptions
             return false;
         }
 
+        // Check channel notification settings
+        const mentioned = message.mention_ids?.includes(user._id);
         switch (this.computeForChannel(message.channel!)) {
             case "muted":
             case "none":
@@ -164,7 +166,12 @@ export default class NotificationOptions
                 return false;
             case "mention":
                 // Ignore if it doesn't mention us.
-                if (!message.mention_ids?.includes(user._id)) return false;
+                if (!mentioned) return false;
+        }
+
+        // Check if we are in focus mode
+        if (user.status?.presence === "Focus" && !mentioned) {
+            return false;
         }
 
         return true;
diff --git a/src/styles/_context-menu.scss b/src/styles/_context-menu.scss
index 1c0cb384..6c0f2f46 100644
--- a/src/styles/_context-menu.scss
+++ b/src/styles/_context-menu.scss
@@ -92,6 +92,10 @@
             background: var(--status-away);
         }
 
+        &.focus {
+            background: var(--status-focus);
+        }
+
         &.busy {
             background: var(--status-busy);
         }
diff --git a/yarn.lock b/yarn.lock
index 9c836b65..a0bf5d13 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -8012,14 +8012,14 @@ __metadata:
   languageName: node
   linkType: hard
 
-"revolt-api@npm:0.5.5-3":
-  version: 0.5.5-3
-  resolution: "revolt-api@npm:0.5.5-3"
+"revolt-api@npm:0.5.5-4":
+  version: 0.5.5-4
+  resolution: "revolt-api@npm:0.5.5-4"
   dependencies:
     "@insertish/oapi": 0.1.18
     axios: ^0.26.1
     lodash.defaultsdeep: ^4.6.1
-  checksum: adf13c87ad22acced931fbac45d49d858346ad03c0aea5045af2e38ee345b85a094b6b2abf76aed93dc6694daae5235e3fd9ed2f213f4700b5ce07688b8a4503
+  checksum: dfb374d58f1b8b5a6de2e7fa05e386b3df3ffb85a450a6894f23c6b9760af8bff0d198d8352063c33084f0dbc6ff84a234300efb0c62c7b27f709887402787f1
   languageName: node
   linkType: hard
 
@@ -8036,7 +8036,7 @@ __metadata:
     lodash.isequal: ^4.5.0
     long: ^5.2.0
     mobx: ^6.3.2
-    revolt-api: 0.5.5-3
+    revolt-api: 0.5.5-4
     ulid: ^2.3.0
     ws: ^8.2.2
   languageName: node

From 6b9106c97506e614aea5d73fad6840da12d9b36b Mon Sep 17 00:00:00 2001
From: Paul Makles <paulmakles@gmail.com>
Date: Sun, 18 Sep 2022 12:32:29 +0100
Subject: [PATCH 2/2] fix: correctly match protocols

---
 src/lib/links.ts | 51 +++++++++++++++++++++++++++++++-----------------
 1 file changed, 33 insertions(+), 18 deletions(-)

diff --git a/src/lib/links.ts b/src/lib/links.ts
index 1a04e14f..211c91d6 100644
--- a/src/lib/links.ts
+++ b/src/lib/links.ts
@@ -1,3 +1,6 @@
+/**
+ * Type of link
+ */
 type LinkType =
     | {
           type: "navigate";
@@ -6,6 +9,9 @@ type LinkType =
     | { type: "external"; href: string; url: URL }
     | { type: "none" };
 
+/**
+ * Allowed origins for relative navigation
+ */
 const ALLOWED_ORIGINS = [
     location.hostname,
     "app.revolt.chat",
@@ -13,27 +19,36 @@ const ALLOWED_ORIGINS = [
     "local.revolt.chat",
 ];
 
+/**
+ * Permissible protocols in URLs
+ */
 const PROTOCOL_WHITELIST = [
-    "https",
-    "ftp",
-    "ftps",
-    "mailto",
-    "news",
-    "irc",
-    "gopher",
-    "nntp",
-    "feed",
-    "telnet",
-    "mms",
-    "rtsp",
-    "svn",
-    "git",
-    "tel",
-    "fax",
-    "xmpp",
-    "magnet",
+    "http:",
+    "https:",
+    "ftp:",
+    "ftps:",
+    "mailto:",
+    "news:",
+    "irc:",
+    "gopher:",
+    "nntp:",
+    "feed:",
+    "telnet:",
+    "mms:",
+    "rtsp:",
+    "svn:",
+    "git:",
+    "tel:",
+    "fax:",
+    "xmpp:",
+    "magnet:",
 ];
 
+/**
+ * Determine what kind of link we are dealing with and sanitise any malicious input
+ * @param href Input URL
+ * @returns Link Type
+ */
 export function determineLink(href?: string): LinkType {
     let internal,
         url: URL | null = null;