Remove border styling from sidebar collapsible sections

[2witstudios]

Summary

This PR removes the top border styling from collapsible sections in the left sidebar and simplifies the Pulse section's header layout by removing the refresh button functionality.

Key Changes

• DashboardFooter, FavoritesSection, RecentsSection, Pulse: Removed className="border-t border-[var(--sidebar-divider)]" from <Collapsible> components
• Skeleton components: Removed border styling from FavoritesSkeleton, PulseSkeleton, and RecentsSkeleton to match the updated collapsible sections
• Pulse section:
  • Removed the refresh button and RefreshCw icon import
  • Simplified the header layout by removing the flex wrapper around the trigger
  • Changed trigger button from flex-1 to w-full for consistent sizing
  • Removed handleRefresh callback and related state management for the refresh functionality

Implementation Details

The border removal appears to be part of a UI refinement to simplify the sidebar's visual hierarchy. The Pulse section changes consolidate the header into a single button trigger, removing the separate refresh action button that was previously displayed alongside the collapse/expand toggle.

https://claude.ai/code/session_01C7S6CJm9ewkthyyhmQdS4c

Summary by CodeRabbit

• Style
  • Removed top divider lines from Dashboard Footer, Favorites, and Recents sections, creating a cleaner, more streamlined sidebar interface.
  • Redesigned the Pulse section header with a unified collapsible toggle button and chevron icon, simplifying the interaction pattern.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

[coderabbitai]

📝 Walkthrough

Walkthrough

Removes top border styling from multiple left sidebar collapsible sections (DashboardFooter, FavoritesSection, RecentsSection) and restructures the Pulse component's header to use a full-width Button wrapped in CollapsibleTrigger instead of a separate icon button with inline refresh handling.

Changes

Cohort / File(s)	Summary
Left Sidebar Border Styling apps/web/src/components/layout/left-sidebar/DashboardFooter.tsx, apps/web/src/components/layout/left-sidebar/FavoritesSection.tsx, apps/web/src/components/layout/left-sidebar/RecentsSection.tsx	Removed top border className from Collapsible components and their skeleton variants, eliminating the divider styling between sidebar sections.
Pulse Component Refactor apps/web/src/components/layout/left-sidebar/Pulse.tsx	Restructured collapsible header from a separate icon button to a CollapsibleTrigger-wrapped full-width Button; removed inline refresh handling and top border styling from both the component and skeleton; chevron now rotates based on collapsed state.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat(ui): make Pulse component collapsible #340: Directly modifies the same Pulse.tsx collapsible header and trigger logic, restructuring the header component layout.
• Refactor sidebar sections to use collapsible components #385: Modifies all the same left-sidebar components (DashboardFooter, FavoritesSection, RecentsSection, Pulse) with related border styling and structure changes.

Poem

🐰 Borders fade like morning dew,
Pulse now triggers clean and true,
Collapsible buttons stretch so wide,
Sidebar dances side by side! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely summarizes the main change: removing border styling from sidebar collapsible sections, which is the primary focus across all modified files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/remove-refresh-dividers-XvpHy

---

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.}