Skip to content

[Fix] Model list does not scroll inside New Session dialog - #2986

Open
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/model-list-trackpad-scroll-0lr01t7rhlmwf
Open

roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/model-list-trackpad-scroll-0lr01t7rhlmwf

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Opened on behalf of @mrubens. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

Make the searchable shared model picker own the active modal layer while open, so wheel and trackpad events reach its portaled list when the picker is nested inside the New Session dialog. Add focused regression coverage for the nested dialog path.

Why this change was made

The dialog scroll lock treated the non-modal model picker portal as outside the dialog content and canceled wheel events, leaving a visibly scrollable long model list stuck at its initial position.

Impact

Long model lists now scroll inside nested modal surfaces without changing search, keyboard selection, or the existing Radix Select path for eight or fewer models. Focused component tests and the repository pre-push checks pass.

Screencasts

Searchable model list scrolling inside New Session

The genuine local New Session surface shows the nested searchable list moving from the initial Claude/DeepSeek rows to later GLM/GPT/Grok rows while the overlay remains stable.

@roomote-community

roomote-community Bot commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed 350314c

@roomote-roomote
roomote-roomote Bot marked this pull request as ready for review September 19, 2026 14:33

This branch has not been deployed

No deployments
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