[fix]侧边栏支持独立滑动

[rootwhois]

fix #29

Summary by CodeRabbit

• Bug Fixes

  • Sidebar now scrolls vertically when its content exceeds the screen height, preventing hidden or inaccessible items and improving navigation on smaller screens or long lists.

• Style

  • Updated sidebar height constraints to respect the viewport and added overflow handling for a more consistent layout and smoother user experience.

[coderabbitai]

Walkthrough

Updates the CategorySidebar container styles to set a maximum height relative to the viewport and enable vertical scrolling when content exceeds available space. No component APIs or behavior beyond scrolling are modified.

Changes

Cohort / File(s)	Summary
Sidebar scroll behavior src/components/CategorySidebar.tsx	Added Tailwind classes max-h-[calc(100vh-8rem)] and overflow-y-auto to make the sidebar vertically scrollable within the viewport height. No other structural or API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

I hopped along the sidebar’s wall,
And found it wouldn’t scroll at all—
Now, viewport winds can safely blow,
The lists cascade, then softly flow.
A happy bun with tidy views,
I scroll, I nibble patch-note news. 🥕

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "[fix]侧边栏支持独立滑动" succinctly and accurately describes the primary change of enabling independent scrolling for the sidebar, matching the changes in the code.
Linked Issues Check	✅ Passed	The pull request adds CSS classes to limit the sidebar height and enable vertical overflow scrolling, directly fulfilling the requirement from issue #29 to allow independent sidebar scrolling.
Out of Scope Changes Check	✅ Passed	The changes are confined to adding scrolling behavior in CategorySidebar.tsx and do not include unrelated modifications, so there are no out-of-scope changes.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8c5f71e and eaefc7f.

📒 Files selected for processing (1)

• src/components/CategorySidebar.tsx (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: build (windows-latest)
• GitHub Check: build (ubuntu-latest)

🔇 Additional comments (1)

src/components/CategorySidebar.tsx (1)

105-105: LGTM — independent sidebar scrolling is correctly implemented. The combination of max-h-[calc(100vh-8rem)], sticky top-24, and overflow-y-auto addresses issue #29. Verify that top-24 (6 rem) matches your header’s actual height and that calc(100vh-8rem) accounts for header plus bottom spacing.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}