Skip to content

fix(orchestration): align Orcastrators "+" right edge with Projects "+" - #34

Merged
zaridan merged 1 commit into
mainfrom
zaridan/fix-orch-orcastrators-plus-align
Jun 24, 2026
Merged

fix(orchestration): align Orcastrators "+" right edge with Projects "+"#34
zaridan merged 1 commit into
mainfrom
zaridan/fix-orch-orcastrators-plus-align

Conversation

@zaridan

@zaridan zaridan commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Problem

The ORCASTRATORS section "+" (New Orcastrator) sat ~8px further left than the Projects "+". Its header row was double-padded: the outer container has px-2 (0.5rem) and the inner header row also had px-2 (0.5rem), so the "+" right edge was 1rem from the sidebar edge. The Projects "+" lives in a single px-2 row, so its right edge is 0.5rem from the edge.

Fix

In OrchestratorsSidebarSection.tsx, change the inner header row's padding from px-2 to pl-2 pr-0 — keep the left padding, drop the right. The "+" right edge now lands at 0.5rem, matching the Projects "+".

  • Outer px-2 untouched → section title left edge and the orchestrator entries below are unchanged.
  • SidebarHeader.tsx, the Button, and the icon are untouched.

Verification

  • pnpm run typecheck — green
  • pnpm run build:desktop — green
  • oxlint / react-doctor / oxfmt on the changed file — green (pre-commit hook passed)
  • Pixel alignment isn't unit-testable; no brittle tests added.

🤖 Generated with Claude Code

The ORCASTRATORS header row was double-padded: the outer container has
px-2 (0.5rem) and the inner header row also had px-2, putting the "+"
right edge 1rem from the sidebar edge — ~8px further left than the
Projects "+", which sits in a single px-2 row (0.5rem inset).

Change the inner header row's right padding from px-2 to pl-2 pr-0 so the
"+" right edge lands at 0.5rem, matching the Projects "+". Title left
position and the entries below are unchanged (outer px-2 untouched).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@zaridan
zaridan merged commit a6899c2 into main Jun 24, 2026
1 check passed
@zaridan
zaridan deleted the zaridan/fix-orch-orcastrators-plus-align branch June 24, 2026 19:29
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