Skip to content

Module sidebar: a white panel beside left-aligned content - #329

Merged
ifahimreza merged 1 commit into
mainfrom
feat/sidebar-panel
Oct 4, 2026
Merged

ifahimreza merged 1 commit into
mainfrom
feat/sidebar-panel

Conversation

@ifahimreza

@ifahimreza ifahimreza commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Fahim, 2026-10-05, on the live CRM and Analytics pages: "how disgusting sidebar look". On the page-grey fixes: "left small sidebar and there no sidebar look bg color it look very big white space". He compared five options in a mockup (https://claude.ai/artifact/MqHubsAeTUzwFjcdKWC2dN) and chose D, sidebar with background, over top tabs (#328).

What was wrong

  • The section's tab row was a white band that started at the sidebar's edge, leaving a grey notch.
  • The content centred itself in the space after the sidebar, so the gap grew with the screen.
  • The "raised" active item, a white box with a ring, read as a text field.
  • The divider was shorter than the items.

What changes

  • The sidebar is a 220px white panel with a #dcdcde hairline on its right, the full height of the page. It joins the white header and the white tab band.
  • The current section takes the page grey #f0f0f1, weight 600, with no ring or shadow. Hover is #f6f7f7 (--s-nav-hover). The divider runs the items' full width.
  • The tab row and the content start 40px after the panel, on the left. The module column is 920px (was 880px, centred).
  • Phone (below 782px) keeps the sideways-scrolling row; the content padding returns to 16px there.
  • admin/DESIGN-ALIGNMENT.md: the palette row, the frame, Navigation and a History entry.

No sibling changes: CRM, Analytics and Rank register the same screens. admin/build/ is not committed.

Checked

  • Jest 336 passed, lint:js clean, build OK.
  • Built in the saddle checkout that saddle.site runs; Fahim looked at it there before asking for the merge.

Not here

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Fahim, 2026-10-05, on the live CRM and Analytics pages: "how disgusting
sidebar look". The tab band started at the sidebar's edge and left a grey
notch, the content centred itself and opened a gap that grew with the
screen, and the "raised" active item read as a text field. Of the
page-grey fixes he said a small sidebar with no background reads as big
empty space, and chose a sidebar with its own background over top tabs
(saddle#328). Mockup: https://claude.ai/artifact/MqHubsAeTUzwFjcdKWC2dN

- The sidebar is a 220px white panel with a hairline on its right, the
  full height, so it joins the white header and the white tab band.
- The current section takes the page grey, weight 600, with no ring or
  shadow. Hover is #f6f7f7. The divider runs the items' full width.
- The tab row and the content start 40px after the panel, on the left.
  The module column is 920px.
- DESIGN-ALIGNMENT.md: the palette row, the frame, Navigation and a
  History entry.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ifahimreza
ifahimreza merged commit 84f9d8e into main Oct 4, 2026
9 checks passed
@ifahimreza ifahimreza mentioned this pull request Oct 5, 2026
6 of 7 tasks
ifahimreza added a commit that referenced this pull request Oct 5, 2026
Version in all five places (header, SADDLE_VERSION, stable tag,
package.json, package-lock.json). The changelog adds the white
sidebar panel (#329) beside the one-page layout (#331). saddle.pot
regenerated: same 2,156 strings, new version and line numbers.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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