Skip to content

Conversation

@ariane-emory
Copy link
Owner

What does this PR do?

How did you verify your code works?

- Add 'center' parameter to moveTo() function for viewport centering
- Use setTimeout to defer scroll until component is rendered
- Pass center=true from filter/current effect for initial positioning
- Keyboard navigation retains edge-scroll behavior for smooth UX

This fixes an issue where the selected item could be outside the visible
viewport when opening dialogs like the session list.
Centers the viewport on the selected item whenever keyboard navigation
(up/down arrows, page up/down) is used in any DialogSelect-based modal.
Mouse hover retains edge-scroll behavior to avoid jumpy interactions.

Applies to: session list, command list, timeline, model selection,
theme list, and all other selection dialogs.
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.

2 participants