Skip to content

PR titles cropped unnecessa...#8466

Merged
alexr00 merged 1 commit intomainfrom
alexr00/issue8453
Feb 3, 2026
Merged

PR titles cropped unnecessa...#8466
alexr00 merged 1 commit intomainfrom
alexr00/issue8453

Conversation

@alexr00
Copy link
Member

@alexr00 alexr00 commented Feb 3, 2026

Fixes #8453

Copilot AI review requested due to automatic review settings February 3, 2026 09:21
@alexr00 alexr00 enabled auto-merge (squash) February 3, 2026 09:21
@alexr00 alexr00 self-assigned this Feb 3, 2026
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 3, 2026
Copy link
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

This PR attempts to fix issue #8453 where PR titles are unnecessarily cropped with ... even when there's enough space to display them. The fix checks the VS Code workbench.list.horizontalScrolling setting to determine whether titles should be truncated.

Changes:

  • Added LIST_HORIZONTAL_SCROLLING constant to settingKeys.ts for the workbench.list.horizontalScrolling setting
  • Modified pullRequestNode.ts to check horizontal scrolling configuration before truncating PR titles

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/common/settingKeys.ts Added constant for workbench.list.horizontalScrolling setting
src/view/treeNodes/pullRequestNode.ts Added logic to check horizontal scrolling setting before truncating PR titles to 50 characters

@alexr00 alexr00 merged commit 3c4f7ed into main Feb 3, 2026
12 checks passed
@alexr00 alexr00 deleted the alexr00/issue8453 branch February 3, 2026 10:02
@catskul
Copy link

catskul commented Feb 3, 2026

Thanks for the quick work! : )

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.

PR titles cropped unnecessa...

4 participants