Skip to content

Add showGroupLabels prop and sidebar storybook stories - #60

Merged
mrabbani merged 1 commit into
mainfrom
feat/sidebar-storybook-stories
Mar 2, 2026
Merged

Add showGroupLabels prop and sidebar storybook stories#60
mrabbani merged 1 commit into
mainfrom
feat/sidebar-storybook-stories

Conversation

@mrabbani

@mrabbani mrabbani commented Mar 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Add showGroupLabels prop to LayoutMenu to toggle group section headers visibility
  • Add HiddenGroupLabels story with boolean Storybook control for showGroupLabels
  • Add ShowHideOnHover story demonstrating sidebar expand/collapse on mouse enter/leave

Test plan

  • Toggle showGroupLabels control in the HiddenGroupLabels story and verify group headers show/hide
  • Hover over collapsed sidebar in ShowHideOnHover story and verify it expands/collapses

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added ability to toggle group labels visibility in the layout menu.
    • Added hover-triggered sidebar auto-expand and collapse functionality.

- Add showGroupLabels prop to LayoutMenu to toggle group section headers
- Add HiddenGroupLabels story with boolean control for showGroupLabels
- Add ShowHideOnHover story demonstrating sidebar expand/collapse on mouse events

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Mar 2, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0ded3b7 and 59b165d.

📒 Files selected for processing (2)
  • src/components/wordpress/Layout.stories.tsx
  • src/components/wordpress/layout-menu.tsx

📝 Walkthrough

Walkthrough

Adds a new showGroupLabels boolean prop to LayoutMenu component to conditionally render group labels, along with two new Storybook stories demonstrating group label visibility control and sidebar hover expand/collapse behavior using the useSidebar hook.

Changes

Cohort / File(s) Summary
Storybook Stories
src/components/wordpress/Layout.stories.tsx
Introduces two new stories: HiddenGroupLabels (with boolean control for showGroupLabels prop) and ShowHideOnHover (demonstrating sidebar expansion/collapse on mouse events via useSidebar hook). Imports useSidebar from the ui/sidebar module.
Layout Menu Component
src/components/wordpress/layout-menu.tsx
Adds showGroupLabels boolean prop to LayoutMenuProps with default value true; gates SidebarGroupLabel rendering based on this prop, allowing group labels to be conditionally hidden.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 A toggle for labels, a hover for grace,
The sidebar now dances in cyberspace!
Stories unfold what the UI can do,
WordPress layouts in rabbit's-eye view! 🥬

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/sidebar-storybook-stories

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

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

@mrabbani
mrabbani merged commit 19174dc into main Mar 2, 2026
1 check failed
@mrabbani
mrabbani deleted the feat/sidebar-storybook-stories branch March 2, 2026 05:59
Aunshon added a commit that referenced this pull request Mar 4, 2026
…h stories (#60)

- Added recharts dependency to package.json
- Created Chart component with ChartContainer, ChartTooltip, ChartTooltipContent, ChartLegend, and ChartLegendContent
- Implemented multiple chart types (Area, Bar, Line, Pie, Radar, Radial) in Chart.stories.tsx
- Updated Card component styles for better layout consistency
- Exported new chart components in index.ts files
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