From aa32adae3d570bbe7f30c13de9d4d1e4da91ef7b Mon Sep 17 00:00:00 2001
From: Pedram Amini <pedram.amini@gmail.com>
Date: Wed, 26 Nov 2025 22:17:29 -0600
Subject: [PATCH] fix: Move shortcuts count from tab header to filter row pill

- Removes count pill from Shortcuts tab to keep all tabs single-line
- Adds count pill before the filter input that shows total or filtered/total
---
 src/renderer/components/SettingsModal.tsx | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/renderer/components/SettingsModal.tsx b/src/renderer/components/SettingsModal.tsx
index 70e35d76..524418c5 100644
--- a/src/renderer/components/SettingsModal.tsx
+++ b/src/renderer/components/SettingsModal.tsx
@@ -535,9 +535,6 @@ export function SettingsModal(props: SettingsModalProps) {
           <button onClick={() => setActiveTab('shortcuts')} className={`px-6 py-4 text-sm font-bold border-b-2 ${activeTab === 'shortcuts' ? 'border-indigo-500' : 'border-transparent'} flex items-center gap-2`} tabIndex={-1}>
             <Keyboard className="w-4 h-4" />
             Shortcuts
-            <span className="text-xs px-1.5 py-0.5 rounded" style={{ backgroundColor: theme.colors.bgActivity, color: theme.colors.textDim }}>
-              {Object.values(props.shortcuts).length}
-            </span>
           </button>
           <button onClick={() => setActiveTab('theme')} className={`px-6 py-4 text-sm font-bold border-b-2 ${activeTab === 'theme' ? 'border-indigo-500' : 'border-transparent'} flex items-center gap-2`} tabIndex={-1}>
             <Palette className="w-4 h-4" />
@@ -1305,7 +1302,10 @@ export function SettingsModal(props: SettingsModalProps) {
 
             return (
               <div className="space-y-3">
-                <div className="flex items-center justify-between">
+                <div className="flex items-center gap-2">
+                  <span className="text-xs px-2 py-1.5 rounded font-medium" style={{ backgroundColor: theme.colors.bgActivity, color: theme.colors.textDim }}>
+                    {shortcutsFilter ? `${filteredCount} / ${totalShortcuts}` : totalShortcuts}
+                  </span>
                   <input
                     ref={shortcutsFilterRef}
                     type="text"
@@ -1315,11 +1315,6 @@ export function SettingsModal(props: SettingsModalProps) {
                     className="flex-1 px-3 py-2 rounded border bg-transparent outline-none text-sm"
                     style={{ borderColor: theme.colors.border, color: theme.colors.textMain }}
                   />
-                  {shortcutsFilter && (
-                    <span className="ml-3 text-xs px-2 py-1 rounded" style={{ backgroundColor: theme.colors.bgActivity, color: theme.colors.textDim }}>
-                      {filteredCount} / {totalShortcuts}
-                    </span>
-                  )}
                 </div>
                 <div className="space-y-2 max-h-[350px] overflow-y-auto pr-2 scrollbar-thin">
                   {filteredShortcuts.map((sc: Shortcut) => (