Skip to content

fix: keep sidebar switcher icons round with long labels#7756

Merged
Dhruwang merged 1 commit into
mainfrom
cursor/fix-sidebar-switcher-icon-shape-66eb
Apr 17, 2026
Merged

fix: keep sidebar switcher icons round with long labels#7756
Dhruwang merged 1 commit into
mainfrom
cursor/fix-sidebar-switcher-icon-shape-66eb

Conversation

@jobenjada
Copy link
Copy Markdown
Member

What does this PR do?

  • Prevents the workspace/organization/account switcher icon circles in the sidebar footer from shrinking when labels are long.
  • Adds shrink-0 to the shared switcherIconClasses in MainNavigation.tsx so icon containers keep a fixed round shape.

Fixes #(issue)

How should this be tested?

  • Open the main sidebar in expanded mode.
  • Use long workspace and organization names.
  • Confirm the gray icon backgrounds in the switcher rows (workspace, organization, account) remain perfectly round.
  • Verify collapsed mode still renders the same icons correctly.

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read How we Code at Formbricks
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand bits
  • Ran pnpm build
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues
  • First PR at Formbricks? Please sign the CLA! Without it we wont be able to merge it 🙏

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the Formbricks Docs if changes were necessary
Open in Web Open in Cursor 

Co-authored-by: Johannes <jobenjada@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@jobenjada jobenjada marked this pull request as ready for review April 16, 2026 13:03
@jobenjada jobenjada requested a review from Dhruwang April 16, 2026 13:03
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: e3ac0c1c-2eed-4094-8b4d-d5d4995af615

📥 Commits

Reviewing files that changed from the base of the PR and between dc1ee72 and 9c6d3f0.

📒 Files selected for processing (1)
  • apps/web/app/(app)/environments/[environmentId]/components/MainNavigation.tsx

Walkthrough

This change adds the shrink-0 Tailwind utility class to the switcherIconClasses styling in the MainNavigation component. The class is applied to an icon wrapper element with flex dimensions of 9 units in both height and width. The modification prevents the icon container from shrinking within its flex parent context.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding shrink-0 to prevent sidebar switcher icons from shrinking with long labels.
Description check ✅ Passed The PR description covers the main sections with sufficient detail: clearly explains what the PR does, provides specific testing instructions, and includes a comprehensive checklist with most required items checked.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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.

Copy link
Copy Markdown
Member

@Dhruwang Dhruwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🚀

@Dhruwang Dhruwang added this pull request to the merge queue Apr 17, 2026
Merged via the queue into main with commit c68f214 Apr 17, 2026
17 checks passed
@Dhruwang Dhruwang deleted the cursor/fix-sidebar-switcher-icon-shape-66eb branch April 17, 2026 08:19
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.

3 participants