Skip to content

improvement(sidebar): allow dragging the sidebar slightly narrower than the default - #7820

Merged
waleedlatif1 merged 2 commits into
stagingfrom
feat/sidebar-resizable-narrower
Sep 14, 2026
Merged

waleedlatif1 merged 2 commits into
stagingfrom
feat/sidebar-resizable-narrower

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Lower SIDEBAR_WIDTH.MIN from 256px to 224px so the sidebar can be dragged slightly narrower; default stays 256px
  • Mirror the new minimum in the pre-paint width script in app/layout.tsx
  • Settings/page navigation resets and the presence-avatar count now anchor to DEFAULT instead of MIN, so their behavior is unchanged

Type of Change

  • Improvement

Testing

Store + resize hook tests pass (added a test for narrowing below the default), type-check, lint, and check:audits pass

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)

🤖 Generated with Claude Code

https://claude.ai/code/session_0118ubb3JgNic3XTBWkoLbVf

@vercel

vercel Bot commented Sep 14, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 14, 2026 9:23pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with the prior sidebar fallback finding fixed and its thread manually resolved.

Summary

This PR lowers the expanded sidebar’s draggable minimum from 256px to 224px while preserving its 256px default behavior.

  • Aligns pre-paint and hydrated-store width clamping, including invalid persisted-width fallbacks.
  • Keeps settings navigation, page navigation, and avatar scaling anchored to the default width.
  • Adds coverage for narrowing below the default and clamping fallback widths on constrained viewports.

Reviews (2) · Last reviewed commit: "fix(sidebar): clamp the default width fa..."

Comment thread apps/sim/app/layout.tsx Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1
waleedlatif1 merged commit cb882ad into staging Sep 14, 2026
25 checks passed
@waleedlatif1
waleedlatif1 deleted the feat/sidebar-resizable-narrower branch September 14, 2026 21:25
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