Skip to content

fix: make Cmd+J worktree jump scroll instant#751

Merged
AmethystLiang merged 1 commit into
mainfrom
better-repo-mode-scroll
Apr 17, 2026
Merged

fix: make Cmd+J worktree jump scroll instant#751
AmethystLiang merged 1 commit into
mainfrom
better-repo-mode-scroll

Conversation

@AmethystLiang

Copy link
Copy Markdown
Contributor

Summary

  • Remove scroll-smooth from the virtualized sidebar viewport so Cmd+J palette reveals land immediately instead of animating the full cross-list distance at browser-default speed.
  • Single-card stepping (Cmd+Shift+↑/↓) also now scrolls instantly — the movement is short enough to feel snappier instant, and it avoids state-machine fragility from toggling scroll-behavior inline (tanstack virtual can issue follow-up scroll corrections after dynamic item measurement).

Test plan

  • Open Cmd+J palette and jump to a worktree far down the list — card should center instantly.
  • Cmd+Shift+↓ / Cmd+Shift+↑ stepping still moves the active selection and keeps the card in view.
  • Cmd+J from a currently-scrolled-away list position still lands on the target.

Remove `scroll-smooth` from the virtualized sidebar viewport so palette
reveals land immediately instead of animating the full cross-list
distance at the browser's default smooth-scroll speed.

Single-card stepping (Cmd+Shift+↑/↓) no longer glides either — the
brevity of the movement makes instant feel snappier, and it avoids
the state-machine risk of toggling `scroll-behavior` inline when
tanstack virtual can also issue follow-up scroll corrections after
dynamic item measurement.
@AmethystLiang
AmethystLiang merged commit d1a3324 into main Apr 17, 2026
1 of 2 checks passed
thomaszdxsn pushed a commit to thomaszdxsn/orca that referenced this pull request Jun 15, 2026
Remove `scroll-smooth` from the virtualized sidebar viewport so palette
reveals land immediately instead of animating the full cross-list
distance at the browser's default smooth-scroll speed.

Single-card stepping (Cmd+Shift+↑/↓) no longer glides either — the
brevity of the movement makes instant feel snappier, and it avoids
the state-machine risk of toggling `scroll-behavior` inline when
tanstack virtual can also issue follow-up scroll corrections after
dynamic item measurement.
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