Skip to content

Conversation

@ethanndickson
Copy link
Member

When navigating slash commands with arrow keys, the dropdown now scrolls to keep the selected item visible. Previously, the selection would move outside the visible area without the view following.

Generated with mux

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@ethanndickson ethanndickson requested a review from Copilot December 9, 2025 07:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves keyboard navigation in the slash command suggestions dropdown by ensuring that the selected item remains visible as users navigate with arrow keys. Previously, keyboard selection could move outside the visible scroll area.

Key changes:

  • Added automatic scrolling to follow keyboard selection in the command suggestions list
  • Implemented a ref-based approach to track and scroll to the selected item

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ethanndickson ethanndickson force-pushed the slash-commands-scrolling-view branch from ef797c0 to 218361f Compare December 9, 2025 07:51
@ethanndickson ethanndickson added this pull request to the merge queue Dec 9, 2025
@ethanndickson ethanndickson removed this pull request from the merge queue due to a manual request Dec 9, 2025
When navigating slash commands with arrow keys, the dropdown now scrolls
to keep the selected item visible. Previously, the selection would move
outside the visible area without the view following.

_Generated with mux_
@ethanndickson ethanndickson force-pushed the slash-commands-scrolling-view branch from 218361f to dcf8ba6 Compare December 9, 2025 07:58
@ethanndickson ethanndickson added this pull request to the merge queue Dec 9, 2025
Merged via the queue into main with commit e91ee5e Dec 9, 2025
19 checks passed
@ethanndickson ethanndickson deleted the slash-commands-scrolling-view branch December 9, 2025 08:10
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