From 4d5d9b4b704de3bdf3b837d9314b8c2e2c664d53 Mon Sep 17 00:00:00 2001
From: Elian Doran <contact@eliandoran.me>
Date: Thu, 19 Feb 2026 18:42:28 +0200
Subject: [PATCH] fix(mobile): some buttons display a + from keyboard shortcut

---
 apps/client/src/widgets/react/Button.tsx | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/apps/client/src/widgets/react/Button.tsx b/apps/client/src/widgets/react/Button.tsx
index 1aab2153e6..6269000563 100644
--- a/apps/client/src/widgets/react/Button.tsx
+++ b/apps/client/src/widgets/react/Button.tsx
@@ -1,13 +1,14 @@
-import type { ComponentChildren, RefObject } from "preact";
-import type { CSSProperties } from "preact/compat";
+import type { ComponentChildren, CSSProperties, RefObject } from "preact";
 import { memo } from "preact/compat";
 import { useMemo } from "preact/hooks";
 
 import { CommandNames } from "../../components/app_context";
-import { isDesktop } from "../../services/utils";
+import { isDesktop, isMobile } from "../../services/utils";
 import ActionButton from "./ActionButton";
 import Icon from "./Icon";
 
+const cachedIsMobile = isMobile();
+
 export interface ButtonProps {
     name?: string;
     /** Reference to the button element. Mostly useful for requesting focus. */
@@ -30,7 +31,7 @@ const Button = memo(({ name, buttonRef, className, text, onClick, keyboardShortc
     // Memoize classes array to prevent recreation
     const classes = useMemo(() => {
         const classList: string[] = ["btn"];
-        
+
         switch(kind) {
             case "primary":
                 classList.push("btn-primary");
@@ -42,7 +43,7 @@ const Button = memo(({ name, buttonRef, className, text, onClick, keyboardShortc
                 classList.push("btn-secondary");
                 break;
         }
-        
+
         if (className) {
             classList.push(className);
         }
@@ -56,7 +57,7 @@ const Button = memo(({ name, buttonRef, className, text, onClick, keyboardShortc
 
     // Memoize keyboard shortcut rendering
     const shortcutElements = useMemo(() => {
-        if (!keyboardShortcut) return null;
+        if (!keyboardShortcut || cachedIsMobile) return null;
         const splitShortcut = keyboardShortcut.split("+");
         return splitShortcut.map((key, index) => (
             <>