Skip to content

Make order history audit details discoverable - #771

Draft
luokerenx4 wants to merge 1 commit into
devfrom
codex/order-history-details-action
Draft

Make order history audit details discoverable#771
luokerenx4 wants to merge 1 commit into
devfrom
codex/order-history-details-action

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

Summary

  • add a visible Details column to order history instead of relying only on whole-row mouse clicks
  • expose per-contract Show/Hide controls with aria-expanded and aria-controls
  • retain the existing row-click convenience while making commit intent and errors keyboard accessible

Evidence

The Demo Portfolio History view showed four static table rows in the accessibility tree. Although each row was clickable with a pointer, there was no focusable control or expanded state, so keyboard and screen-reader users could not reach the commit hash, intent message, rejection error, or resolution timestamp hidden behind the row.

Verification

  • pnpm vitest run ui/src/pages/UTADetailPage.order-history.spec.tsx (1 passed)
  • cd ui && npx tsc -b
  • npx tsc --noEmit
  • pnpm test (3389 passed, 9 skipped)
  • real Demo route: AMD exposes Show details for AMD order; activating it changes to expanded Hide details for AMD order and reveals f31c9a2 / Add AMD ahead of earnings; AAPL, NVDA, and GOOG expose their own controls

@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 11:06pm

Request Review

@luokerenx4 luokerenx4 added area:trading UTA, orders, positions, portfolio, or trading review theme:accessibility Keyboard, assistive technology, or interaction semantics workflow:parallel Autonomous parallel contribution; leave open for later acceptance labels Jul 29, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:trading UTA, orders, positions, portfolio, or trading review theme:accessibility Keyboard, assistive technology, or interaction semantics workflow:parallel Autonomous parallel contribution; leave open for later acceptance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant