Skip to content

fix(settings): align general appearance dropdowns to a uniform width#5045

Closed
waleedlatif1 wants to merge 1 commit into
stagingfrom
fix/settings-dropdown-align
Closed

fix(settings): align general appearance dropdowns to a uniform width#5045
waleedlatif1 wants to merge 1 commit into
stagingfrom
fix/settings-dropdown-align

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • The three appearance dropdowns in General settings were different widths: Theme and Snap-to-grid (ChipSelect) hugged their content (~90px), while Timezone (ChipCombobox) was pinned to 260px — so they read as a ragged column.
  • Give all three a shared 240px trigger (via fullWidth + a common width wrapper constant), so they line up as one column with flush edges. Dropdown menus now match their triggers.

Type of Change

  • Bug fix (visual alignment)

Scope / impact

  • Touches only general.tsx. Uses existing ChipSelect props (fullWidth, dropdownWidth) — the shared ChipSelect/ChipCombobox components are not modified, so no other page is affected.
  • No behavioral change: Timezone keeps search; all options and handlers are unchanged. The 5 Switch toggles in the panel are untouched.

Testing

Tested manually in General settings (light + dark).

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)

Theme/Snap-to-grid (ChipSelect) hugged their content (~90px) while Timezone
(ChipCombobox) was pinned to 260px, so the three read as a ragged column. Give
all three a shared 240px trigger via fullWidth + a common width wrapper so they
align as one column; menus match their triggers. No behavioral change — timezone
keeps search, options and handlers are untouched.
@vercel

vercel Bot commented Jun 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 (UTC)
docs Skipped Skipped Jun 14, 2026 4:57pm

Request Review

@cursor

cursor Bot commented Jun 14, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
UI-only layout in general.tsx with no settings logic or shared component changes.

Overview
Theme, Timezone, and Snap to grid in General settings now share a 240px trigger column so the preference row no longer looks ragged.

Adds DROPDOWN_TRIGGER_CLASS (w-[240px] flex-shrink-0) and wraps all three controls in it. Theme and Snap to grid ChipSelect triggers use fullWidth and dropdownWidth='trigger' instead of content-hugging widths; Timezone ChipCombobox moves from a 260px wrapper to the same 240px width with dropdownWidth={240}. Handlers and options are unchanged.

Reviewed by Cursor Bugbot for commit 992d552. Configure here.

@greptile-apps

greptile-apps Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes a visual alignment issue in the General settings panel where the three appearance dropdowns (Theme, Timezone, Snap-to-grid) rendered at inconsistent widths. All three are now aligned to a shared 240px trigger width via a module-level DROPDOWN_TRIGGER_CLASS constant.

  • Theme and Snap-to-grid ChipSelect components are wrapped in a w-[240px] flex-shrink-0 div and given fullWidth + dropdownWidth='trigger' so their menus stretch to match the trigger exactly.
  • Timezone ChipCombobox wrapper is narrowed from 260px to 240px, and its explicit dropdownWidth is updated to match.

Confidence Score: 5/5

Safe to merge — purely cosmetic alignment change scoped to a single file with no logic or behavioral modification.

The change wraps existing, unmodified chip components in a fixed-width container and passes valid, documented props. Both ChipSelect and Combobox already handle these props correctly. The only functional delta is that the Timezone dropdown menu shrinks from 260px to 240px, which is intentional and consistent with the new unified width.

No files require special attention — the single changed file is self-contained and affects only the General settings appearance section.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/settings/components/general/general.tsx Wraps Theme and Snap-to-grid ChipSelects in a 240px container with fullWidth/dropdownWidth='trigger', and aligns the Timezone ChipCombobox wrapper from 260px to 240px — purely visual alignment, no logic or behavior changes.

Reviews (1): Last reviewed commit: "fix(settings): align general appearance ..." | Re-trigger Greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

Folding this into #5044 per review — the timezone setting drives the scheduled-tasks calendar, so the dropdown-alignment fix rides along there. Same change cherry-picked onto that branch.

@waleedlatif1 waleedlatif1 deleted the fix/settings-dropdown-align branch June 15, 2026 17:08
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