feat: add sidebar scroll position persistence#75
Conversation
- Add `useScroll` hook with debounced scroll event handling - Add `useScrollToElement` hook to save/restore scroll position via localStorage and in-memory NavigationStateContext - Add `NavigationStateProvider` context for session-level state - Update `Sidebar` component to use `useScrollToElement` with a `useLayoutEffect` workaround (SideBar lacks forwardRef support)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Updates Reviewed by Cursor Bugbot for commit 93020dd. Bugbot is set up for automated code reviews on this repo. Configure here. |
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/nodejs-website Please review the changes when you have a chance. Thank you! 🙏 |
…gationStateProvider
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 305c7d0. Configure here.

useScrollhook with debounced scroll event handlinguseScrollToElementhook to save/restore scroll position via localStorage and in-memory NavigationStateContextNavigationStateProvidercontext for session-level stateSidebarcomponent to useuseScrollToElementwith auseLayoutEffectworkaround (SideBar lacks forwardRef support)#74 fix sidebar scroll position persistence