Fix sidebar focus

src-web/components/Sidebar.tsx

@@ -90,17 +90,13 @@ export function Sidebar({ className }: Props) {
     namespace: NAMESPACE_NO_SYNC,
   });
 
-  useHotKey(
-    'http_request.duplicate',
-    async () => {
+  useHotKey('http_request.duplicate', async () => {
     if (activeRequest?.model === 'http_request') {
       await duplicateHttpRequest.mutateAsync();
     } else {
       await duplicateGrpcRequest.mutateAsync();
     }
-    },
-    { enable: !hidden },
-  );
+  });
 
   const isCollapsed = useCallback(
     (id: string) => collapsed.value?.[id] ?? false,
@@ -178,13 +174,14 @@ export function Sidebar({ className }: Props) {
       const children = tree?.children ?? [];
       const id =
         forced?.id ?? children.find((m) => m.item.id === activeRequest?.id)?.item.id ?? null;
+
+      setHasFocus(true);
+      setSelectedId(id);
+      setSelectedTree(tree);
+
       if (id == null) {
         return;
       }
-
-      setSelectedId(id);
-      setSelectedTree(tree);
-      setHasFocus(true);
       if (!noFocusSidebar) {
         sidebarRef.current?.focus();
       }
@@ -246,9 +243,8 @@ export function Sidebar({ className }: Props) {
   useKeyPressEvent('Backspace', handleDeleteKey);
   useKeyPressEvent('Delete', handleDeleteKey);
 
-  useHotKey(
-    'sidebar.focus',
-    async () => {
+  useHotKey('sidebar.focus', async () => {
+    console.log('sidebar.focus', { hidden, hasFocus });
     // Hide the sidebar if it's already focused
     if (!hidden && hasFocus) {
       await hide();
@@ -266,9 +262,7 @@ export function Sidebar({ className }: Props) {
         ? { forced: { id: selectedId, tree: selectedTree } }
         : undefined,
     );
-    },
-    { enable: !hidden },
-  );
+  });
 
   useKeyPressEvent('Enter', (e) => {
     if (!hasFocus) return;

src-web/hooks/useHotKey.ts

@@ -83,8 +83,7 @@ export function useHotKey(
       }
 
       const key = normalizeKey(e.key, os);
-
-      currentKeys.current.add(normalizeKey(key, os));
+      currentKeys.current.add(key);
 
       for (const [hkAction, hkKeys] of Object.entries(hotkeys) as [HotkeyAction, string[]][]) {
         for (const hkKey of hkKeys) {
@@ -108,7 +107,7 @@ export function useHotKey(
         return;
       }
       const key = normalizeKey(e.key, os);
-      currentKeys.current.delete(normalizeKey(key, os));
+      currentKeys.current.delete(key);
 
       // Clear all keys if no longer holding modifier
       // HACK: This is to get around the case of DOWN SHIFT -> DOWN : -> UP SHIFT -> UP ;