From 42771f7137b0502a49a57e99ab04b14a511032c6 Mon Sep 17 00:00:00 2001
From: 3nt3 <gott@3nt3.de>
Date: Tue, 4 Jan 2022 12:03:39 +0100
Subject: [PATCH] feat: make kick and ban red in the context menu (#350)

Co-authored-by: Paul Makles <paulmakles@gmail.com>
---
 src/lib/ContextMenus.tsx | 45 ++++++++++++++++++++++++++--------------
 1 file changed, 30 insertions(+), 15 deletions(-)

diff --git a/src/lib/ContextMenus.tsx b/src/lib/ContextMenus.tsx
index cb657c20..e3bf4a27 100644
--- a/src/lib/ContextMenus.tsx
+++ b/src/lib/ContextMenus.tsx
@@ -459,15 +459,18 @@ export default function ContextMenus() {
                         locale?: string,
                         disabled?: boolean,
                         tip?: Children,
+                        color?: string,
                     ) {
                         lastDivider = false;
                         elements.push(
                             <MenuItem data={action} disabled={disabled}>
-                                <Text
-                                    id={`app.context_menu.${
-                                        locale ?? action.action
-                                    }`}
-                                />
+                                <span style={{color}}>
+                                    <Text
+                                        id={`app.context_menu.${
+                                            locale ?? action.action
+                                        }`}
+                                    />
+                                </span>
                                 {tip && <div className="tip">{tip}</div>}
                             </MenuItem>,
                         );
@@ -652,18 +655,30 @@ export default function ContextMenus() {
                             if (
                                 serverPermissions & ServerPermission.KickMembers
                             )
-                                generateAction({
-                                    action: "kick_member",
-                                    target: server,
-                                    user: user!,
-                                });
+                                generateAction(
+                                    {
+                                        action: "kick_member",
+                                        target: server,
+                                        user: user!,
+                                    },
+                                    undefined, // this is needed because generateAction uses positional, not named parameters
+                                    undefined,
+                                    null,
+                                    "var(--error)", // the only relevant part really
+                                );
 
                             if (serverPermissions & ServerPermission.BanMembers)
-                                generateAction({
-                                    action: "ban_member",
-                                    target: server,
-                                    user: user!,
-                                });
+                                generateAction(
+                                    {
+                                        action: "ban_member",
+                                        target: server,
+                                        user: user!,
+                                    },
+                                    undefined,
+                                    undefined,
+                                    null,
+                                    "var(--error)",
+                                );
                         }
                     }