fix(web): right panel shortcuts on non-Latin layouts and terminal close dialog UX#13428

@@ -75,22 +170,15 @@ export function ConfirmDialogHost() {
         if (!open) completeConfirmDialogClose();
       }}
     >
-      <AlertDialogPopup>
-        <AlertDialogHeader>
-          <AlertDialogTitle>{copy.title}</AlertDialogTitle>
-          {copy.description ? (
-            <AlertDialogDescription className="whitespace-pre-line">
-              {copy.description}
-            </AlertDialogDescription>
-          ) : null}
-        </AlertDialogHeader>
-        <AlertDialogFooter>
-          <AlertDialogClose render={<Button variant="outline" />}>Cancel</AlertDialogClose>
-          <Button variant={confirmVariant} onClick={onConfirm}>
-            Confirm
-          </Button>
-        </AlertDialogFooter>
-      </AlertDialogPopup>
+      {isMounted ? (
+        <ConfirmDialogBody
+          key={state.message}

@@ -32,8 +37,20 @@ export async function confirmTerminalClose(
               .map((label) => `"${label}"`)
               .join(", ")}.`,
           ].join("\n"),
-      { variant: "destructive" },
+      {
+        variant: "destructive",
+        checkbox: {
+          label: "Don't ask again",
+          onCheckedChange: (checked) => {
+            dontAskAgain = checked;
+          },
+        },
+      },
     );
+    if (confirmed && dontAskAgain) {
+      void persistClientSettingsPatch({ confirmTerminalClose: false });
+    }
+    return confirmed;

@@ -246,17 +246,23 @@ function tabAudioState(overlay: DesktopPreviewOverlay | null): TabAudioState {
 type SurfaceShortcutEvent = Pick<
   KeyboardEvent,
   "altKey" | "ctrlKey" | "defaultPrevented" | "isComposing" | "key" | "metaKey"
->;
+> & { readonly code?: string };
 
 export function surfaceShortcutActionForKey<
   const Action extends { available: boolean; shortcut: string },
 >(actions: readonly Action[], event: SurfaceShortcutEvent): Action | null {
   if (event.defaultPrevented || event.isComposing) return null;
   if (event.metaKey || event.ctrlKey || event.altKey) return null;
+  const layoutKey = event.key.toLowerCase();
+  const letterCode = event.code?.match(/^Key([A-Z])$/)?.[1]?.toLowerCase();
   return (
-    actions.find(
-      (action) => action.available && action.shortcut.toLowerCase() === event.key.toLowerCase(),
-    ) ?? null
+    actions.find((action) => {
+      if (!action.available) return false;
+      const targetShortcut = action.shortcut.toLowerCase();
+      if (targetShortcut === layoutKey) return true;
+      if (letterCode && !/^[a-z]$/.test(layoutKey) && targetShortcut === letterCode) return true;

@@ -32,8 +37,23 @@ export async function confirmTerminalClose(
               .map((label) => `"${label}"`)
               .join(", ")}.`,
           ].join("\n"),
-      { variant: "destructive" },
+      {
+        variant: "destructive",
+        checkbox: {
+          label: "Don't ask again",
+          onCheckedChange: (checked) => {
+            dontAskAgain = checked;
+          },
+        },
+      },
     );
+    if (confirmed && dontAskAgain) {
+      await persistClientSettingsUpdate((settings) => ({
+        ...settings,
+        confirmTerminalClose: false,
+      }));
+    }
+    return confirmed;
   } catch {

@@ -621,6 +621,9 @@ export function useSettingsRestore(onRestored?: () => void) {
       ...(settings.confirmThreadDelete !== DEFAULT_UNIFIED_SETTINGS.confirmThreadDelete
         ? ["Delete confirmation"]
         : []),
+      ...(settings.confirmTerminalClose !== DEFAULT_UNIFIED_SETTINGS.confirmTerminalClose
+        ? ["Terminal close confirmation"]
+        : []),

@@ -421,6 +428,19 @@ function RightPanelEmptyState(props: {
   const highlightIndex =
     availableActions.length === 0 ? -1 : Math.min(highlight, availableActions.length - 1);
 
+  // Letter shortcuts work while the launcher is visible, not only while it
+  // is focused; focus moves around too easily (stray clicks) to carry them.
+  // Capture phase so app-level key handlers cannot swallow the event first;
+  // typing contexts and already-handled events are left alone.
+  const isPanelOpen = props.open !== false;
+  const containerRef = useRef<HTMLDivElement | null>(null);
+
+  useEffect(() => {
+    if (isPanelOpen) {
+      containerRef.current?.focus();
+    }
+  }, [isPanelOpen, shortcutActionsRef]);

@@ -1405,6 +1422,7 @@ export function RightPanelTabs(props: RightPanelTabsProps) {
       <div className="flex min-h-0 flex-1 flex-col" data-right-panel-surface-content>
         {props.activeSurfaceId === null ? (
           <RightPanelEmptyState
+            open={props.open}

@@ -75,22 +170,15 @@ export function ConfirmDialogHost() {
         if (!open) completeConfirmDialogClose();
       }}
     >
-      <AlertDialogPopup>
-        <AlertDialogHeader>
-          <AlertDialogTitle>{copy.title}</AlertDialogTitle>
-          {copy.description ? (
-            <AlertDialogDescription className="whitespace-pre-line">
-              {copy.description}
-            </AlertDialogDescription>
-          ) : null}
-        </AlertDialogHeader>
-        <AlertDialogFooter>
-          <AlertDialogClose render={<Button variant="outline" />}>Cancel</AlertDialogClose>
-          <Button variant={confirmVariant} onClick={onConfirm}>
-            Confirm
-          </Button>
-        </AlertDialogFooter>
-      </AlertDialogPopup>
+      {isMounted ? (
+        <ConfirmDialogBody
+          key={state.message}