Skip to content

fix(sidebar): keep the collapsed profile chip inside the rail - #6282

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/collapsed-profile-hover
Aug 5, 2026
Merged

fix(sidebar): keep the collapsed profile chip inside the rail#6282
waleedlatif1 merged 1 commit into
stagingfrom
fix/collapsed-profile-hover

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Collapsed, the profile chip's hover fill ran the full width of the rail and got clipped at its edge — the label stays in the layout (the rail hides it with opacity, not display, so the fade survives a toggle), and its empty box still contributed the content row's gap, putting the chip's automatic minimum at 38px against a 35px rail
  • Floored the collapsed chip at min-w-0 so it fills exactly the rail, and the avatar keeps the same 8px offset as the help glyph above it

Type of Change

  • Bug fix

Testing

Typecheck + lint clean; not yet verified in a running browser

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 5, 2026 4:51am

Request Review

@cursor

cursor Bot commented Aug 5, 2026

Copy link
Copy Markdown

PR Summary

Cursor Bugbot is generating a summary for commit 0cc5332. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR constrains the collapsed sidebar profile chip to the available rail width while preserving the expanded chip behavior.

  • Adds min-w-0 only to the collapsed full-width profile trigger.
  • Documents how the retained, opacity-hidden label affected the chip’s automatic minimum width.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete functional or accessibility regression identified.

The change is limited to the collapsed trigger’s minimum-width constraint; its border box and hover fill now fit the padded rail while the non-shrinking avatar retains its intended offset.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/sidebar-footer/sidebar-footer.tsx The collapsed profile trigger now permits flex shrinking to the footer’s 35px inner width without changing expanded behavior or avatar alignment.

Reviews (1): Last reviewed commit: "fix(sidebar): keep the collapsed profile..." | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit 020ec68 into staging Aug 5, 2026
24 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/collapsed-profile-hover branch August 5, 2026 04:55
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