Skip to content

Update xterm and integrate new scroll bars#221500

Merged
Tyriar merged 10 commits intomainfrom
tyriar/scrollbar
Jul 11, 2024
Merged

Update xterm and integrate new scroll bars#221500
Tyriar merged 10 commits intomainfrom
tyriar/scrollbar

Conversation

@Tyriar
Copy link
Copy Markdown
Contributor

@Tyriar Tyriar commented Jul 11, 2024

Fixes #211199
Fixes #216521
See xtermjs/xterm.js#5096

The change is fairly big due to the way that the viewport is dealt with is quite different and vscode had a bunch of customizations to the old way to get it to bottom align, fill horizontal space, etc.

Wins:

Changes:

  • This removes the 2px padding on the very bottom of the terminal in order for the scroll bar to extend all the way to the bottom of the panel. This was a workaround to help display underscores in the terminal under certain fonts when using the DOM renderer. Let's fix issues as they come as this was a hack that degraded the default (webgl) UX.

Before:

image

After:

image

@Tyriar Tyriar added this to the July 2024 milestone Jul 11, 2024
@Tyriar Tyriar self-assigned this Jul 11, 2024
@Tyriar Tyriar marked this pull request as ready for review July 11, 2024 18:58
meganrogge
meganrogge previously approved these changes Jul 11, 2024
meganrogge
meganrogge previously approved these changes Jul 11, 2024
@Tyriar
Copy link
Copy Markdown
Contributor Author

Tyriar commented Jul 11, 2024

Integration test failed again, weird. Surprised this PR could impact that one

@Tyriar Tyriar enabled auto-merge July 11, 2024 23:11
@Tyriar Tyriar merged commit 6d7dc00 into main Jul 11, 2024
@Tyriar Tyriar deleted the tyriar/scrollbar branch July 11, 2024 23:30
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Aug 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

3 participants