Skip to content

Optimize sidebar vertical spacing - #65

Merged
shaohaozhi286 merged 7 commits into
masterfrom
sidebar-optimization
Sep 21, 2026
Merged

shaohaozhi286 merged 7 commits into
masterfrom
sidebar-optimization

Conversation

@LIUXIN557

Copy link
Copy Markdown
Collaborator

Summary

  • normalize ACECode sidebar vertical spacing without changing horizontal alignment contracts
  • keep tree rows on the fixed 24px / minmax(0,1fr) / 76px grid with existing padding and right alignment
  • add a token-free standalone preview page for visual verification
  • document the implementation and acceptance criteria

Validation

  • node src/lib/sidebarAlignmentArchitecture.test.js
  • pnpm test
  • pnpm build (passed before the preview-only follow-up commit)
  • git diff --cached --check

Notes

The branch contains only the sidebar spacing implementation, its test/style/documentation changes, and the standalone preview page. Existing unrelated working-tree changes were intentionally left uncommitted.

@shaohaozhi286
shaohaozhi286 merged commit 398f906 into master Sep 21, 2026
3 checks passed
shaohaozhi286 pushed a commit that referenced this pull request Sep 22, 2026
This reverts commit 398f906 against its first parent, restoring sidebar layout, workspace actions, menus, onboarding, and the four-dot running indicator. Preserve later changes and refresh restored UI translations.
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.

2 participants