Skip to content

Improve market sidebar keyboard search - #758

Draft
luokerenx4 wants to merge 1 commit into
devfrom
codex/market-sidebar-keyboard
Draft

Improve market sidebar keyboard search#758
luokerenx4 wants to merge 1 commit into
devfrom
codex/market-sidebar-keyboard

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

Summary

  • let users move between federated Market sidebar search providers with Arrow Up/Down and open the highlighted exact provider with Enter
  • make Escape clear the inline search and keep mouse hover in sync with the keyboard highlight
  • give the sidebar search input an explicit accessible name and add focused regression coverage

Verification

  • pnpm vitest run ui/src/components/MarketSidebar.spec.tsx (3 passed)
  • npx tsc --noEmit
  • cd ui && npx tsc -b
  • pnpm test (358 files passed, 1 skipped; 3391 tests passed, 9 skipped)
  • Demo browser at /market: expanded the Market sidebar, searched apple, pressed Arrow Down, confirmed the alpaca-paper row highlight, pressed Enter, and confirmed AAPL opened with alpaca-paper · AAPL (iex) selected
  • git diff --check

Boundary touch

  • none; UI-only navigation and accessibility behavior

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview, Comment Jul 28, 2026 9:26pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant