Skip to content

Refactor UI spacing for improved consistency#305667

Merged
mrleemurray merged 9 commits intomainfrom
mrleemurray/legal-tan-snake
Mar 27, 2026
Merged

Refactor UI spacing for improved consistency#305667
mrleemurray merged 9 commits intomainfrom
mrleemurray/legal-tan-snake

Conversation

@mrleemurray
Copy link
Copy Markdown
Contributor

Adjust margins and padding across various UI components to enhance layout consistency and visual appeal. This includes updates to card-like components, footers, and interactive elements. The changes aim to create a more cohesive user experience.

Copilot AI review requested due to automatic review settings March 27, 2026 14:26
@mrleemurray mrleemurray self-assigned this Mar 27, 2026
@vs-code-engineering vs-code-engineering bot added this to the Backlog milestone Mar 27, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Refactors spacing (margins/padding) across the Agent Sessions window UI to improve visual consistency, including card-like part margins, list/container padding, and footer spacing.

Changes:

  • Standardize card margins for chat bar / auxiliary bar / panel and align related inner paddings.
  • Adjust padding/margins in sessions lists, welcome, toolbars, and changes/CI widgets to reduce horizontal inset.
  • Hide the account update container when the update button is hidden, and tweak the Changes tree configuration for spacing.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/vs/sessions/contrib/sessions/browser/media/sessionsViewPane.css Adjusts new-session button container padding to match updated spacing.
src/vs/sessions/contrib/sessions/browser/media/sessionsList.css Increases horizontal padding for the sessions list scroll area.
src/vs/sessions/contrib/sessions/browser/media/customizationsToolbar.css Tweaks toolbar margin to align with the new 10px spacing baseline.
src/vs/sessions/contrib/chat/browser/media/chatWelcomePart.css Reduces welcome part horizontal padding for consistency.
src/vs/sessions/contrib/changes/browser/media/ciStatusWidget.css Removes/adjusts horizontal padding to make CI rows/header align with the new layout.
src/vs/sessions/contrib/changes/browser/media/changesView.css Updates header padding to reduce horizontal inset.
src/vs/sessions/contrib/changes/browser/changesView.ts Sets tree options (notably indent) to change visual spacing of the changes tree.
src/vs/sessions/contrib/accountMenu/browser/media/accountWidget.css Hides update container when the update button is hidden (via :has).
src/vs/sessions/browser/parts/sidebarPart.ts Reduces footer bottom margin constant to match new footer spacing.
src/vs/sessions/browser/parts/panelPart.ts Updates margin constants used for panel card-like layout.
src/vs/sessions/browser/parts/media/sidebarPart.css Updates sidebar footer padding/margins and inset border alignment.
src/vs/sessions/browser/parts/chatBarPart.ts Updates chat bar margin constants used for layout sizing.
src/vs/sessions/browser/parts/auxiliaryBarPart.ts Updates auxiliary bar margin constants used for layout sizing.
src/vs/sessions/browser/media/style.css Updates card margins/paddings and starting-style animation margins for the new spacing baseline.

Comment thread src/vs/sessions/contrib/changes/browser/changesView.ts Outdated
Comment thread src/vs/sessions/browser/media/style.css
Comment thread src/vs/sessions/contrib/accountMenu/browser/media/accountWidget.css Outdated
mrleemurray and others added 2 commits March 27, 2026 14:35
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…t.css

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mrleemurray mrleemurray marked this pull request as ready for review March 27, 2026 14:37
lszomoru
lszomoru previously approved these changes Mar 27, 2026
@mrleemurray mrleemurray enabled auto-merge March 27, 2026 14:38
@mrleemurray mrleemurray merged commit 9685f8d into main Mar 27, 2026
18 checks passed
@mrleemurray mrleemurray deleted the mrleemurray/legal-tan-snake branch March 27, 2026 15: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.

4 participants