Skip to content

Sessions: Remove line-height from changes-decoration-badge styles in changesView.css#306275

Merged
mrleemurray merged 1 commit intomainfrom
mrleemurray/compatible-silver-yak
Mar 30, 2026
Merged

Sessions: Remove line-height from changes-decoration-badge styles in changesView.css#306275
mrleemurray merged 1 commit intomainfrom
mrleemurray/compatible-silver-yak

Conversation

@mrleemurray
Copy link
Copy Markdown
Contributor

Eliminate the line-height property from button styles to improve visual consistency in the changes view. Testing involves checking the appearance of buttons in the changes view to ensure they render correctly without the line-height adjustment.

Copilot AI review requested due to automatic review settings March 30, 2026 09:53
@mrleemurray mrleemurray enabled auto-merge (squash) March 30, 2026 09:53
@mrleemurray mrleemurray self-assigned this Mar 30, 2026
@vs-code-engineering vs-code-engineering bot added this to the Backlog milestone Mar 30, 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

Removes a CSS line-height declaration in the Sessions “Changes” view stylesheet to adjust visual alignment/consistency.

Changes:

  • Removed line-height: 1 from the .changes-decoration-badge styling in changesView.css.

@mrleemurray mrleemurray changed the title Sessions: Remove line-height from button styles in changesView.css Sessions: Remove line-height from changes-decoration-badge styles in changesView.css Mar 30, 2026
@mrleemurray mrleemurray merged commit 3d5035e into main Mar 30, 2026
22 checks passed
@mrleemurray mrleemurray deleted the mrleemurray/compatible-silver-yak branch March 30, 2026 10:19
mrleemurray added a commit that referenced this pull request Mar 30, 2026
…changesView.css (#306275)

style: remove line-height property from button styles in changesView.css

Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.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.

3 participants