Skip to content

fix(sidebar): align spacing and tooltip offset with design#788

Merged
rohanchkrabrty merged 1 commit intomainfrom
worktree-sidebar
May 7, 2026
Merged

fix(sidebar): align spacing and tooltip offset with design#788
rohanchkrabrty merged 1 commit intomainfrom
worktree-sidebar

Conversation

@rohanchkrabrty
Copy link
Copy Markdown
Contributor

@rohanchkrabrty rohanchkrabrty commented May 7, 2026 •
edited
Loading

Summary

  • Tooltip sideOffset set to 16px on collapsed Sidebar.Item and Sidebar.More triggers
  • Increased .main gap between sidebar groups from --rs-space-2 (4px) to --rs-space-6 (20px)
  • Removed margin-top from .nav-group and .nav-group-header (and the dead :first-child reset)
  • Removed justify-content: space-between from .nav-group-header
  • Docs: added a "Group with icon" example showing leadingIcon passed to Sidebar.Group

@vercel
Copy link
Copy Markdown

vercel Bot commented May 7, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
apsara Ready Ready Preview, Comment May 7, 2026 6:57am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026 •
edited
Loading

Warning

Rate limit exceeded

@rohanchkrabrty has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 21 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 86dd6e0f-c45a-4b1a-9b0f-3724141ddc35

📥 Commits

Reviewing files that changed from the base of the PR and between e6b07ef and e1c195a.

📒 Files selected for processing (6)
  • apps/www/src/content/docs/components/sidebar/demo.ts
  • apps/www/src/content/docs/components/sidebar/index.mdx
  • docs/V1-migration.md
  • packages/raystack/components/sidebar/sidebar-item.tsx
  • packages/raystack/components/sidebar/sidebar-more.tsx
  • packages/raystack/components/sidebar/sidebar.module.css

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.

@rohanchkrabrty rohanchkrabrty self-assigned this May 7, 2026
@rohanchkrabrty @claude
fix(sidebar): align spacing and tooltip offset with design
e1c195a 
- Tooltip sideOffset of 16px on collapsed Sidebar.Item and Sidebar.More triggers
- Increased gap between sidebar groups from 4px to 20px
- Removed margin-top from .nav-group and .nav-group-header (and the dead :first-child reset)
- Removed justify-content: space-between from .nav-group-header

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rohanchkrabrty rohanchkrabrty merged commit 536f021 into main May 7, 2026
5 checks passed
@rohanchkrabrty rohanchkrabrty deleted the worktree-sidebar branch May 7, 2026 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Shreyag02 Shreyag02 Shreyag02 approved these changes

@rohilsurana rohilsurana Awaiting requested review from rohilsurana

@rsbh rsbh Awaiting requested review from rsbh

@paanSinghCoder paanSinghCoder Awaiting requested review from paanSinghCoder

Assignees

@rohanchkrabrty rohanchkrabrty

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rohanchkrabrty @Shreyag02