forked from molecule-ai/molecule-core
b1a1c8e4a9
Two small fixes for the batch-action toolbar: 1. The deselect button's title says "Clear selection (Escape)" — but pressing Escape did NOTHING. The title has been lying since the bar shipped. Now wired: window keydown handler calls clearSelection when Esc fires. Skipped while the confirm dialog is open (`pending !== null`) so the dialog's own Esc-cancels takes precedence, and skipped during a busy in-flight action so the user can't strand a partial-failure mid-flight. 2. focus-visible:ring-zinc-500/70 → focus-visible:ring-accent/50 on the deselect button. The hardcoded zinc broke the semantic- token pattern used by the other action buttons. Tests: two new vitest cases — Esc clears with selection, Esc no-op when empty (the bar isn't mounted at count===0 so the listener never registers). Full suite: 1222/1222. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| e2e | ||
| public | ||
| src | ||
| .env.example | ||
| .gitignore | ||
| components.json | ||
| Dockerfile | ||
| next.config.ts | ||
| package-lock.json | ||
| package.json | ||
| playwright.config.ts | ||
| playwright.staging.config.ts | ||
| postcss.config.js | ||
| tsconfig.json | ||
| vitest.config.ts | ||