Skip to content

feat(app): inspector polish — loading states, responsive, keyboard nav#44

Merged
sepehr-safari merged 1 commit into
mainfrom
feat/app-inspector-polish
Jul 8, 2026
Merged

feat(app): inspector polish — loading states, responsive, keyboard nav#44
sepehr-safari merged 1 commit into
mainfrom
feat/app-inspector-polish

Conversation

@sepehr-safari

Copy link
Copy Markdown
Member

Summary

Polishes the Inspector UI with loading states, responsive layout, and keyboard navigation.

Closes #29

Changes

Loading states

  • Spinner with "Parsing trace…" indicator during analysis
  • Analyze button shows "Analyzing…" and disables while parsing
  • Uses setTimeout(0) to let the loading state render before heavy synchronous parsing

Responsive layout (mobile-friendly)

  • Flex-wrap for sample scenario buttons on small screens
  • Smaller text sizes on mobile (text-xs → text-sm/sm:text-base breakpoints)
  • Adaptive grid: grid-cols-2 sm:grid-cols-4 for stat cards
  • Shorter textarea on mobile (h-48 sm:h-64)
  • Smaller padding on mobile (p-3 sm:p-4)

Keyboard navigation

  • Arrow up/down moves through events in the timeline
  • Selected event auto-scrolls into view
  • Focus container has tabIndex={0} for keyboard focus
  • Hint text shows "↑↓ to navigate"

Other polish

  • Sticky header for better UX on long traces
  • Error state: improved guidance for empty input
  • All states (loading, empty, error) properly handled and hidden during loading

Verification

  • pnpm lint — ✅
  • pnpm typecheck — ✅
  • pnpm test — ✅ (196 tests)
  • pnpm build — ✅
  • pnpm format:check — ✅

- Loading state: spinner with parsing indicator, button disables
- Responsive layout: mobile-friendly (flex-wrap, smaller text, adaptive grids)
- Keyboard navigation: arrow up/down to move through events in the timeline
- Sticky header for better UX on long traces
- Error state: improved guidance for empty input
- All states (loading, empty, error) properly handled

Closes #29
@sepehr-safari sepehr-safari added this to the v0.1.0 — Inspector MVP milestone Jul 7, 2026
@sepehr-safari sepehr-safari added type:feature New feature or enhancement package:app apps/web — single Next.js app labels Jul 7, 2026
@sepehr-safari sepehr-safari merged commit 6db4905 into main Jul 8, 2026
1 check passed
@sepehr-safari sepehr-safari deleted the feat/app-inspector-polish branch July 8, 2026 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package:app apps/web — single Next.js app type:feature New feature or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(app): inspector failure summary + report export + polish

1 participant