Skip to content

Polish ACP agent selector scrolling#685

Closed
Finesssee wants to merge 1 commit into
athasdev:masterfrom
Finesssee:codex/acp-selector-scroll-ux
Closed

Polish ACP agent selector scrolling#685
Finesssee wants to merge 1 commit into
athasdev:masterfrom
Finesssee:codex/acp-selector-scroll-ux

Conversation

@Finesssee
Copy link
Copy Markdown
Contributor

Summary

  • Make the ACP agent selector taller and scrollable enough for the expanded provider catalog
  • Render provider icon URLs with colored presentation and keep the existing provider icon fallback
  • Keep keyboard selection anchored in view while moving through the full provider list

Validation

  • bun typecheck
  • bunx vp check src/features/ai/components/selectors/agent-selector.tsx
  • git diff --check
  • Verified in Athas Dev with Computer Use: expanded provider list is visible, colored icons render, and tail providers like VT Code, Qwen Code, and siGit Code are reachable through search

@Finesssee
Copy link
Copy Markdown
Contributor Author

Superseded by #682. I folded the selector scroll polish into the ACP runtime/catalog branch so the selector file no longer needs a separate PR that conflicts with the runtime work.

@Finesssee
Copy link
Copy Markdown
Contributor Author

Closing as superseded by #682, which now includes this selector polish on top of the ACP runtime/catalog changes.

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