gloomberb 0.8.1 → 0.9.0

package/src/components/layout/shell/index.test.tsx

@@ -61,10 +61,11 @@ function createShellPluginRegistry(options?: {
   } as unknown as PluginRegistry;
 }
 
-async function emitKeypress(event: { name?: string; sequence?: string; ctrl?: boolean; meta?: boolean; shift?: boolean }) {
+async function emitKeypress(event: { name?: string; sequence?: string; ctrl?: boolean; meta?: boolean; shift?: boolean; alt?: boolean }) {
   await act(async () => {
     const keyEvent = {
       ctrl: false,
+      alt: false,
       meta: false,
       option: false,
       shift: false,
@@ -312,6 +313,7 @@ describe("Shell", () => {
     const frame = testSetup.captureCharFrame();
     expect(frame).toContain("Settings");
     expect(frame).toContain("Ctrl+,");
+    expect(frame).not.toContain("Layout Actions");
     expect(frame).not.toContain("CmdOrCtrl");
   });
 
@@ -400,24 +402,92 @@ describe("Shell", () => {
     });
     await testSetup.renderOnce();
 
-    expect(testSetup.captureCharFrame()).toContain("Dock Pane");
+    const frame = testSetup.captureCharFrame();
+    expect(frame).toContain("Dock Pane");
+    expect(frame).not.toContain("Layout Actions");
   });
 
   test("resolves pane management shortcuts", () => {
     const base = { ctrl: false, meta: true, super: true, shift: true, alt: false };
     expect(resolvePaneManagementShortcut({ ...base, name: ",", key: ",", shift: false })).toBe("settings");
     expect(resolvePaneManagementShortcut({ ...base, name: "w", key: "w", ctrl: true, meta: false, super: false, shift: false })).toBe("close");
+    expect(resolvePaneManagementShortcut({ ...base, name: "w", key: "w", shift: false, alt: true })).toBe("close-all-floating");
+    expect(resolvePaneManagementShortcut({ ...base, name: "W", key: "W", shift: false })).toBeNull();
     expect(resolvePaneManagementShortcut({ ...base, name: "D", key: "D" })).toBe("toggle-floating");
     expect(resolvePaneManagementShortcut({ ...base, name: "o", key: "o" })).toBe("pop-out");
     expect(resolvePaneManagementShortcut({ ...base, name: "c", key: "c" })).toBe("copy-screenshot");
     expect(resolvePaneManagementShortcut({ ...base, name: "l", key: "l" })).toBe("layout-actions");
     expect(resolvePaneManagementShortcut({ ...base, name: "g", key: "g" })).toBe("gridlock-all");
     expect(resolvePaneManagementShortcut({ ...base, name: "m", key: "m" })).toBe("window-mode");
+    expect(resolvePaneManagementShortcut({ ...base, name: "r", key: "r" })).toBe("window-resize-mode");
     expect(resolvePaneManagementShortcut({ ...base, name: "n", key: "n" })).toBeNull();
     expect(resolvePaneManagementShortcut({ ...base, name: "d", key: "d", alt: true })).toBeNull();
     expect(resolvePaneManagementShortcut({ ...base, name: "d", key: "d", meta: false, super: false })).toBeNull();
   });
 
+  test("exits window mode on Enter without changes", async () => {
+    const config = createDefaultConfig("/tmp/gloomberb-shell-window-mode-resize-shortcut-test");
+    const mainPane = requireLayoutInstance(config, "portfolio-list:main");
+    const detailPane = requireLayoutInstance(config, "ticker-detail:main");
+    const dockedLayout = {
+      dockRoot: {
+        kind: "split" as const,
+        axis: "horizontal" as const,
+        ratio: 0.5,
+        first: { kind: "pane" as const, instanceId: "portfolio-list:main" },
+        second: { kind: "pane" as const, instanceId: "ticker-detail:main" },
+      },
+      instances: [{ ...mainPane }, { ...detailPane }],
+      floating: [],
+      detached: [],
+    };
+    const { actions } = await renderShellForWindowModeTest(
+      createShellStateWithLayout(config, dockedLayout, "portfolio-list:main"),
+    );
+
+    await emitKeypress({ name: "m", ctrl: true, shift: true });
+    expect(testSetup.captureCharFrame()).toContain("WINDOW MOVE");
+
+    await emitKeypress({ name: "enter" });
+    await act(async () => {
+      await testSetup!.renderOnce();
+    });
+    expect(testSetup.captureCharFrame()).not.toContain("WINDOW MOVE");
+    expect(actions.some((action) => action.type === "UPDATE_LAYOUT")).toBe(false);
+  });
+
+  test("starts resize mode directly from the resize shortcut", async () => {
+    const config = createDefaultConfig("/tmp/gloomberb-shell-window-mode-resize-shortcut-test");
+    const mainPane = requireLayoutInstance(config, "portfolio-list:main");
+    const detailPane = requireLayoutInstance(config, "ticker-detail:main");
+    const dockedLayout = {
+      dockRoot: {
+        kind: "split" as const,
+        axis: "horizontal" as const,
+        ratio: 0.5,
+        first: { kind: "pane" as const, instanceId: "portfolio-list:main" },
+        second: { kind: "pane" as const, instanceId: "ticker-detail:main" },
+      },
+      instances: [{ ...mainPane }, { ...detailPane }],
+      floating: [],
+      detached: [],
+    };
+    const { actions } = await renderShellForWindowModeTest(
+      createShellStateWithLayout(config, dockedLayout, "portfolio-list:main"),
+    );
+
+    await emitKeypress({ name: "r", ctrl: true, shift: true });
+    expect(testSetup.captureCharFrame()).toContain("WINDOW RESIZE");
+
+    await emitKeypress({ name: "enter" });
+    await act(async () => {
+      await testSetup!.renderOnce();
+    });
+    const frame = testSetup.captureCharFrame();
+    expect(frame).not.toContain("WINDOW RESIZE");
+    expect(actions.some((action) => action.type === "UPDATE_LAYOUT")).toBe(false);
+  });
+
   test("moves a floating pane in window mode and commits once", async () => {
     const config = createDefaultConfig("/tmp/gloomberb-shell-window-mode-test");
     const floatingLayout = cloneLayout(config.layout);
@@ -785,4 +855,52 @@ describe("Shell", () => {
     expect(updateLayout?.layout.floating).toEqual([]);
   });
 
+  test("closes all floating panes with Ctrl+Alt+W", async () => {
+    const config = createDefaultConfig("/tmp/gloomberb-shell-test");
+    const mainPane = config.layout.instances.find((instance) => instance.instanceId === "portfolio-list:main");
+    const firstDetailPane = config.layout.instances.find((instance) => instance.instanceId === "ticker-detail:main");
+    if (!mainPane || !firstDetailPane) throw new Error("missing default panes");
+    const secondDetailPane = {
+      ...firstDetailPane,
+      instanceId: "ticker-detail:secondary",
+    };
+
+    const mixedLayout = {
+      dockRoot: { kind: "pane" as const, instanceId: "portfolio-list:main" },
+      instances: [{ ...mainPane }, { ...firstDetailPane }, secondDetailPane],
+      floating: [
+        { instanceId: "ticker-detail:main", x: 4, y: 2, width: 30, height: 8 },
+        { instanceId: "ticker-detail:secondary", x: 8, y: 3, width: 30, height: 8 },
+      ],
+      detached: [],
+    };
+    const state = {
+      ...createInitialState({
+        ...config,
+        layout: cloneLayout(mixedLayout),
+        layouts: [{ name: "Default", layout: cloneLayout(mixedLayout) }],
+      }),
+      focusedPaneId: "portfolio-list:main",
+    };
+    const actions: Array<any> = [];
+
+    testSetup = await testRender(
+      <AppContext value={{ state, dispatch: (action) => actions.push(action) }}>
+        <TestDialogProvider>
+          <Shell pluginRegistry={createShellPluginRegistry()} />
+        </TestDialogProvider>
+      </AppContext>,
+      { width: 40, height: 12 },
+    );
+
+    await testSetup.renderOnce();
+    await emitKeypress({ name: "w", ctrl: true, alt: true });
+
+    const updateLayout = actions.find((action) => action.type === "UPDATE_LAYOUT");
+    expect(actions).toContainEqual({ type: "PUSH_LAYOUT_HISTORY" });
+    expect(updateLayout?.layout.instances.map((instance: { instanceId: string }) => instance.instanceId)).toEqual(["portfolio-list:main"]);
+    expect(updateLayout?.layout.floating).toEqual([]);
+    expect(updateLayout?.layout.dockRoot).toEqual({ kind: "pane", instanceId: "portfolio-list:main" });
+  });
+
 });

package/src/components/layout/shell/index.tsx

@@ -186,6 +186,7 @@ export function Shell({
   });
 
   const {
+    closeAllFloatingPanes,
     closeFocusedPane,
     copyFocusedPaneScreenshot,
     copyPaneScreenshot,
@@ -213,6 +214,7 @@ export function Shell({
 
   useShellPaneManagementShortcuts({
     cancelActiveDrag,
+    closeAllFloatingPanes,
     closeFocusedPane,
     copyFocusedPaneScreenshot,
     focusedPaneId,

package/src/components/layout/shell/menu.ts

@@ -101,15 +101,9 @@ export function menuForPane(
     {
       id: "window-resize-mode",
       label: "Resize Window...",
-      accelerator: "WIN resize",
+      accelerator: PANE_MANAGEMENT_ACCELERATORS.windowResizeMode,
       onSelect: () => pluginRegistry.openWindowMode(pane.instance.instanceId, "resize"),
     },
-    {
-      id: "layout-actions",
-      label: "Layout Actions...",
-      accelerator: PANE_MANAGEMENT_ACCELERATORS.layoutActions,
-      onSelect: () => pluginRegistry.openCommandBar("LAY "),
-    },
   );
 
   return baseActions;

package/src/components/layout/shell/pane/actions.ts

@@ -5,6 +5,7 @@ import {
   floatPane,
   gridlockAllPanes,
   isPaneInLayout,
+  removeFloatingPanes,
   removePane,
   type ResolvedPane,
 } from "../../../../plugins/pane-manager";
@@ -73,6 +74,12 @@ export function useShellPaneActions({
     return true;
   }, [focusedPaneId, persistLayout, visibleLayout]);
 
+  const closeAllFloatingPanes = useCallback(() => {
+    if (visibleLayout.floating.length === 0) return false;
+    persistLayout(removeFloatingPanes(visibleLayout));
+    return true;
+  }, [persistLayout, visibleLayout]);
+
   const copyFocusedPaneScreenshot = useCallback(() => {
     if (!focusedPaneId || !nativePaneChrome || !rendererHost.copyPngImage) return false;
     if (!paneMap.has(focusedPaneId)) return false;
@@ -115,6 +122,7 @@ export function useShellPaneActions({
   }, [persistLayout, visibleLayout]);
 
   return {
+    closeAllFloatingPanes,
     closeFocusedPane,
     copyFocusedPaneScreenshot,
     copyPaneScreenshot,

package/src/components/layout/shell/pane/management-shortcuts.ts

@@ -1,5 +1,6 @@
 import { useEffect, useRef } from "react";
 import { useShortcut } from "../../../../react/input";
+import type { WindowEditMode } from "../../../../plugins/registry";
 import {
   createDoubleEscapeCloseState,
   recordDoubleEscapeClose,
@@ -12,6 +13,7 @@ import {
 
 interface ShellPaneManagementShortcutOptions {
   cancelActiveDrag(): void;
+  closeAllFloatingPanes(): boolean;
   closeFocusedPane(): boolean;
   copyFocusedPaneScreenshot(): boolean;
   focusedPaneId: string | null;
@@ -22,12 +24,13 @@ interface ShellPaneManagementShortcutOptions {
   openLayoutMenu(): void;
   overlayOpen: boolean;
   popOutFocusedPane(): boolean;
-  startWindowMode(): void;
+  startWindowMode(paneId?: string, mode?: WindowEditMode): void;
   toggleFocusedPaneFloating(): boolean;
 }
 
 export function useShellPaneManagementShortcuts({
   cancelActiveDrag,
+  closeAllFloatingPanes,
   closeFocusedPane,
   copyFocusedPaneScreenshot,
   focusedPaneId,
@@ -82,6 +85,9 @@ export function useShellPaneManagementShortcuts({
       case "close":
         handled = closeFocusedPane();
         break;
+      case "close-all-floating":
+        handled = closeAllFloatingPanes();
+        break;
       case "settings":
         handled = openFocusedPaneSettings();
         break;
@@ -102,7 +108,11 @@ export function useShellPaneManagementShortcuts({
         handled = gridlockVisiblePanes();
         break;
       case "window-mode":
-        startWindowMode();
+        startWindowMode(undefined, "move");
+        handled = true;
+        break;
+      case "window-resize-mode":
+        startWindowMode(undefined, "resize");
         handled = true;
         break;
     }

package/src/components/layout/shell/shortcuts.ts

@@ -6,9 +6,11 @@ export const PANE_MANAGEMENT_ACCELERATORS = {
   popOut: "CmdOrCtrl+Shift+O",
   copyScreenshot: "CmdOrCtrl+Shift+C",
   close: "CmdOrCtrl+W",
+  closeAllFloating: "CmdOrCtrl+Alt+W",
   layoutActions: "CmdOrCtrl+Shift+L",
   gridlockAll: "CmdOrCtrl+Shift+G",
   windowMode: "CmdOrCtrl+Shift+M",
+  windowResizeMode: "CmdOrCtrl+Shift+R",
 } as const;
 
 export type PaneManagementShortcut =
@@ -17,24 +19,30 @@ export type PaneManagementShortcut =
   | "pop-out"
   | "copy-screenshot"
   | "close"
+  | "close-all-floating"
   | "layout-actions"
   | "gridlock-all"
-  | "window-mode";
+  | "window-mode"
+  | "window-resize-mode";
 
 export function resolvePaneManagementShortcut(
   event: Pick<KeyEventLike, "name" | "key" | "ctrl" | "meta" | "super" | "shift" | "alt">,
 ): PaneManagementShortcut | null {
   if (!event.ctrl && !event.meta && !event.super) return null;
-  const name = (event.name ?? event.key ?? "").toLowerCase();
-  if (name === "w") return "close";
-  if (!event.shift && name === ",") return "settings";
-  if (!event.shift || event.alt) return null;
+  const rawName = event.name ?? event.key ?? "";
+  const name = rawName.toLowerCase();
+  const shifted = event.shift || rawName !== name;
+  if (!shifted && event.alt && name === "w") return "close-all-floating";
+  if (!shifted && name === "w") return "close";
+  if (!shifted && name === ",") return "settings";
+  if (!shifted || event.alt) return null;
   if (name === "c") return "copy-screenshot";
   if (name === "d") return "toggle-floating";
   if (name === "o") return "pop-out";
   if (name === "l") return "layout-actions";
   if (name === "g") return "gridlock-all";
   if (name === "m") return "window-mode";
+  if (name === "r") return "window-resize-mode";
   return null;
 }
 
@@ -42,6 +50,6 @@ export function inputCaptureAllowsPaneManagementShortcut(
   shortcut: PaneManagementShortcut,
   event: Pick<KeyEventLike, "meta" | "super" | "targetEditable">,
 ): boolean {
-  if (shortcut !== "close") return false;
+  if (shortcut !== "close" && shortcut !== "close-all-floating") return false;
   return event.meta || event.super || event.targetEditable !== true;
 }

package/src/components/layout/shell/window-mode/index.ts

@@ -128,9 +128,13 @@ export function useShellWindowMode({
     if (!windowMode) return;
     const committedLayout = resolveWindowEditCommitLayout(windowMode, bounds, dockGeometryOptions);
     const hasPendingCommit = windowEditHasPendingCommit(windowMode, bounds, dockGeometryOptions);
-    if (hasPendingCommit) {
-      persistLayout(committedLayout);
+    if (!hasPendingCommit) {
+      focusPane(windowMode.paneId);
+      setWindowMode(null);
+      return;
     }
+
+    persistLayout(committedLayout);
     focusPane(windowMode.paneId);
     setWindowMode({
       paneId: windowMode.paneId,
@@ -138,7 +142,7 @@ export function useShellWindowMode({
       mode: "move",
       focus: normalizeWindowEditFocus({ kind: "move" }, committedLayout, windowMode.paneId, "move", bounds, dockGeometryOptions),
       dirty: false,
-      notice: hasPendingCommit ? "Committed" : "No changes",
+      notice: "Committed",
     });
   }, [bounds, dockGeometryOptions, focusPane, persistLayout, windowMode]);
 
@@ -166,6 +170,7 @@ export function useShellWindowMode({
 
   useShortcut((event) => {
     if (!windowMode) return;
+    if (event.ctrl || event.meta || event.super || event.alt) return;
     const name = (event.name ?? event.key ?? "").toLowerCase();
     let handled = true;
 

package/src/components/layout/window-edit/presentation.ts

@@ -73,14 +73,14 @@ export function windowEditStatusLine(
 export function windowEditHelpText(state: WindowEditState): string {
   if (state.mode === "move") {
     if (state.focus.kind === "dock-move") {
-      return "arrows/hjkl choose side  m retarget  Tab/w window  d float  r resize  Enter commit  Esc exit/cancel";
+      return "arrows/hjkl choose side  m retarget  Tab/w window  d float  r resize  Enter commit/exit  Esc exit/cancel";
     }
-    return "Tab/w window  d dock/float  r resize  arrows/hjkl target/move  Shift fast  Enter commit  Esc exit/cancel";
+    return "Tab/w window  d dock/float  r resize  arrows/hjkl target/move  Shift fast  Enter commit/exit  Esc exit/cancel";
   }
   if (state.focus.kind === "floating-resize") {
-    return "Tab corner  w window  m move  arrows/hjkl resize  Shift fast  Enter commit  Esc exit/cancel";
+    return "Tab corner  w window  m move  arrows/hjkl resize  Shift fast  Enter commit/exit  Esc exit/cancel";
   }
-  return "arrows/hjkl move divider  Tab divider  w window  m move  Shift fast  Enter commit  Esc exit/cancel";
+  return "arrows/hjkl move divider  Tab divider  w window  m move  Shift fast  Enter commit/exit  Esc exit/cancel";
 }
 
 export function resolveWindowEditDockMovePreview(

package/src/components/onboarding/onboarding-steps.tsx

@@ -101,7 +101,8 @@ export function ShortcutsStep({ pluginRegistry }: { pluginRegistry: PluginRegist
   const shortcutDisplayMode = getShortcutDisplayMode(uiHost.kind);
   const platformShortcut = (keys: string | readonly string[]) => formatPrimaryShortcut(keys, shortcutPlatform, shortcutDisplayMode);
   const keyboardShortcuts = [
-    { key: "Ctrl+P / `", desc: "Open the command bar" },
+    { key: "Ctrl+P", desc: "Open command mode" },
+    { key: "`", desc: "Open ticker search" },
     { key: "Tab", desc: "Switch between panels" },
     { key: platformShortcut("W"), desc: "Close the focused pane" },
     { key: platformShortcut(["Shift", "D"]), desc: "Dock or float the focused pane" },
@@ -155,7 +156,7 @@ export function ShortcutsStep({ pluginRegistry }: { pluginRegistry: PluginRegist
       </Box>
       <Box height={1} />
       <Box height={1}>
-        <Text fg={colors.textDim}>{"Type these in the command bar ("}<Span fg={colors.text}><Strong>{"Ctrl+P"}</Strong></Span>{" or "}<Span fg={colors.text}><Strong>{"`"}</Strong></Span>{"):"}</Text>
+        <Text fg={colors.textDim}>{"Type these in the command bar ("}<Span fg={colors.text}><Strong>{"Ctrl+P"}</Strong></Span>{"):"}</Text>
       </Box>
       <Box height={1} />
 

package/src/components/pane-settings-dialog/field-dialogs.tsx

@@ -61,7 +61,6 @@ export function TuiSelectFieldDialog(props: SelectFieldDialogProps) {
   return (
     <DialogFrame
       title={field.label}
-      footer="Use ↑↓ to choose · enter to apply · esc cancel"
     >
       <ListView
         items={field.options.map((option) => ({
@@ -158,7 +157,6 @@ function TuiTextFieldDialog(props: TextFieldDialogProps) {
   return (
     <DialogFrame
       title={field.label}
-      footer="Enter to apply · esc cancel"
     >
       <Box flexDirection="column" gap={1}>
         <TextField

package/src/components/pane-settings-dialog/tui.tsx

@@ -29,7 +29,7 @@ export function TuiPaneSettingsDialogBody({
   onActivate: (field: PaneSettingField | undefined) => void;
 }) {
   return (
-    <DialogFrame title={title} footer="Use ↑↓ to choose · enter to edit · esc cancel">
+    <DialogFrame title={title}>
       <ListView
         items={fields.map((field) => ({
           id: field.key,

package/src/components/ticker/list-table-view.tsx

@@ -3,7 +3,6 @@ import {
   useEffect,
   useMemo,
   useRef,
-  useState,
   type ReactNode,
   type RefObject,
 } from "react";
@@ -61,8 +60,6 @@ export interface TickerListTableViewProps {
   setHoveredIdx?: (index: number | null) => void;
   keyboardNavigation?: boolean;
   onRootKeyDown?: (event: DataTableKeyEvent) => boolean | void;
-  onSelectIndex?: (index: number, ticker: TickerRecord) => void;
-  onActivateIndex?: (index: number, ticker: TickerRecord) => void;
   onVisibleRangeChange?: (range: TickerListVisibleRange) => void;
   visibleRangeBuffer?: number;
   resetScrollKey?: unknown;
@@ -84,12 +81,18 @@ const FLASHABLE_QUOTE_COLUMN_IDS = new Set([
   "bid",
   "ask",
   "spread",
+  "spread_pct",
   "ext_hours",
   "market_cap",
+  "volume",
+  "dollar_volume",
+  "range_52w",
   "mkt_value",
+  "weight",
   "day_pnl",
   "pnl",
   "pnl_pct",
+  "mark_delta",
 ]);
 
 const EMPTY_FLASH_SYMBOLS = new Map<string, QuoteFlashDirection>();
@@ -120,8 +123,6 @@ export function TickerListTableView({
   setHoveredIdx,
   keyboardNavigation = true,
   onRootKeyDown,
-  onSelectIndex,
-  onActivateIndex,
   onVisibleRangeChange,
   visibleRangeBuffer = 0,
   resetScrollKey,
@@ -153,8 +154,6 @@ export function TickerListTableView({
     () => tickers.findIndex((ticker) => ticker.metadata.ticker === cursorSymbol),
     [cursorSymbol, tickers],
   );
-  const [scrollToIndex, setScrollToIndex] = useState<number | null>(null);
-  const [scrollToIndexVersion, setScrollToIndexVersion] = useState(0);
   const lastScrollCursorSymbolRef = useRef<string | null | undefined>(undefined);
 
   const emitVisibleRange = useCallback(() => {
@@ -177,16 +176,12 @@ export function TickerListTableView({
   useEffect(() => {
     const shouldScrollToCursor = lastScrollCursorSymbolRef.current !== cursorSymbol;
     lastScrollCursorSymbolRef.current = cursorSymbol;
-    if (shouldScrollToCursor && selectedIndex >= 0) {
-      setScrollToIndex(selectedIndex);
-      setScrollToIndexVersion((current) => current + 1);
-    }
-    queueMicrotask(emitVisibleRange);
+    if (shouldScrollToCursor && selectedIndex >= 0) queueMicrotask(emitVisibleRange);
   }, [cursorSymbol, emitVisibleRange, selectedIndex]);
 
   useEffect(() => {
     queueMicrotask(emitVisibleRange);
-  }, [emitVisibleRange, scrollToIndexVersion]);
+  }, [emitVisibleRange]);
 
   const renderCell = useCallback((
     ticker: TickerRecord,
@@ -236,18 +231,16 @@ export function TickerListTableView({
   }, [financialsMap, renderer, showContextMenu]);
 
   const handleRowMouseDown = useCallback((ticker: TickerRecord, _index: number, event: TableMouseEvent) => {
-    setCursorSymbol(ticker.metadata.ticker);
     if (event.button !== 2) return false;
     if (nativeContextMenu !== true) {
       showTickerContextMenu(ticker, event);
     }
     return true;
-  }, [nativeContextMenu, setCursorSymbol, showTickerContextMenu]);
+  }, [nativeContextMenu, showTickerContextMenu]);
 
   const handleRowContextMenu = useCallback((ticker: TickerRecord, _index: number, event: TableMouseEvent) => {
-    setCursorSymbol(ticker.metadata.ticker);
     showTickerContextMenu(ticker, event);
-  }, [setCursorSymbol, showTickerContextMenu]);
+  }, [showTickerContextMenu]);
 
   return (
     <DataTableView<TickerRecord, ColumnConfig>
@@ -258,9 +251,13 @@ export function TickerListTableView({
       sortDirection={sortDirection}
       onHeaderClick={onHeaderClick ?? (() => {})}
       getItemKey={(ticker) => ticker.metadata.ticker}
-      isSelected={(ticker) => ticker.metadata.ticker === cursorSymbol}
-      onSelect={(ticker) => {
-        setCursorSymbol(ticker.metadata.ticker);
+      selection={{
+        kind: "id",
+        selectedId: cursorSymbol,
+        getId: (ticker) => ticker.metadata.ticker,
+        onChange: (symbol) => {
+          setCursorSymbol(symbol);
+        },
       }}
       onActivate={(ticker) => {
         onRowActivate?.(ticker);
@@ -273,9 +270,6 @@ export function TickerListTableView({
       emptyStateHint={emptyHint}
       virtualize={virtualize}
       overscan={overscan}
-      selectedIndex={selectedIndex}
-      onSelectIndex={onSelectIndex}
-      onActivateIndex={onActivateIndex}
       rootBefore={rootBefore}
       rootAfter={rootAfter}
       rootWidth={rootWidth}
@@ -290,8 +284,6 @@ export function TickerListTableView({
       keyboardNavigation={keyboardNavigation}
       onRootKeyDown={onRootKeyDown}
       resetScrollKey={resetScrollKey}
-      scrollToIndex={scrollToIndex}
-      scrollToIndexVersion={scrollToIndexVersion}
     />
   );
 }

package/src/components/ui/choice-dialog.tsx

@@ -17,6 +17,7 @@ export interface ChoiceDialogChoice {
 export interface ChoiceDialogProps extends PromptContext<string> {
   title: string;
   choices: ChoiceDialogChoice[];
+  selectedChoiceId?: string;
   footer?: string;
   bgColor?: string;
 }
@@ -30,14 +31,23 @@ function getChoiceDescription(choice: ChoiceDialogChoice | undefined): string {
   return choice?.description ?? "";
 }
 
+function getInitialChoiceIndex(choices: ChoiceDialogChoice[], selectedChoiceId: string | undefined): number {
+  if (!selectedChoiceId) return 0;
+  const selectedIndex = choices.findIndex((choice) => choice.id === selectedChoiceId);
+  return selectedIndex >= 0 ? selectedIndex : 0;
+}
+
 export function ChoiceDialog({
   resolve,
   title,
   choices,
-  footer = "↑↓ choose · Enter/click select · Esc cancel",
+  selectedChoiceId,
+  footer,
   bgColor = colors.bg,
 }: ChoiceDialogProps) {
-  const [index, setIndex] = useState(() => clampChoiceIndex(0, choices.length));
+  const [index, setIndex] = useState(() =>
+    clampChoiceIndex(getInitialChoiceIndex(choices, selectedChoiceId), choices.length)
+  );
   const selectedIndex = clampChoiceIndex(index, choices.length);
   const selectedChoice = selectedIndex >= 0 ? choices[selectedIndex] : undefined;
   const items = useMemo<ListViewItem[]>(() => choices.map((choice) => ({

package/src/components/ui/multi-select/dialog.tsx

@@ -2,7 +2,7 @@ import { Box, Text, useUiHost } from "../../../ui";
 import { TextAttributes } from "../../../ui";
 import { useShortcut } from "../../../react/input";
 import { type AlertContext, useDialog, useDialogKeyboard } from "../../../ui/dialog";
-import { type ReactNode, useEffect, useMemo, useState } from "react";
+import { type ReactNode, useEffect, useMemo, useRef, useState } from "react";
 import { colors } from "../../../theme/colors";
 import { isDetailBackNavigationKey } from "../../../utils/back-navigation";
 import { isPlainKey } from "../../../utils/keyboard";
@@ -253,6 +253,7 @@ export function MultiSelectDialogButton({
 }: MultiSelectDialogButtonProps) {
   const isDesktopWeb = useUiHost().kind === "desktop-web";
   const dialog = useDialog();
+  const triggerMouseDownRef = useRef(false);
   const summary = summarizeMultiSelectValues({ options, selectedValues, emptyLabel });
   const buttonLabel = `${label}: ${summary}`;
   const buttonText = ` ${buttonLabel} `;
@@ -311,6 +312,17 @@ export function MultiSelectDialogButton({
     );
   }
 
+  const startTriggerPress = (event?: DialogTriggerEvent) => {
+    triggerMouseDownRef.current = true;
+    stopMouseEvent(event);
+  };
+  const finishTriggerPress = (event?: DialogTriggerEvent) => {
+    const startedOnTrigger = triggerMouseDownRef.current;
+    triggerMouseDownRef.current = false;
+    if (startedOnTrigger) openDialog(event);
+    else stopMouseEvent(event);
+  };
+
   return (
     <Box
       id={idPrefix ? `${idPrefix}:button` : undefined}
@@ -318,14 +330,14 @@ export function MultiSelectDialogButton({
       width={buttonText.length}
       flexDirection="row"
       backgroundColor={disabled ? colors.panel : colors.selected}
-      onMouseDown={stopMouseEvent}
-      onMouseUp={openDialog}
+      onMouseDown={startTriggerPress}
+      onMouseUp={finishTriggerPress}
     >
       <Text
         fg={disabled ? colors.textMuted : colors.selectedText}
         attributes={TextAttributes.BOLD}
-        onMouseDown={stopMouseEvent}
-        onMouseUp={openDialog}
+        onMouseDown={startTriggerPress}
+        onMouseUp={finishTriggerPress}
       >
         {buttonText}
       </Text>