-
Notifications
You must be signed in to change notification settings - Fork 28.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrated terminal: sticky scroll header has wider letter spacing #216817
Labels
bug
Issue identified by VS Code Team member as probable bug
insiders-released
Patch has been released in VS Code Insiders
terminal-rendering
Milestone
Comments
This is because they use different renderers, see #218427 |
Is my understanding correct that the goal is to make the sticky scroll header the same size & width as the rest of the terminal (after #218427)? Having a different width/size does not work well with stuff like ZSH right prompt. |
aaronchucarroll
pushed a commit
to aaronchucarroll/vscode
that referenced
this issue
Jul 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Issue identified by VS Code Team member as probable bug
insiders-released
Patch has been released in VS Code Insiders
terminal-rendering
Type: Bug
Configuration using all defaults except these:
Compare the non-sticky appearance:
![Screenshot 2024-06-21 at 10 55 00](https://private-user-images.githubusercontent.com/11131775/341804079-e183b93d-d6de-467e-b4a5-309fde4fb661.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3NDI2MDMsIm5iZiI6MTcyMDc0MjMwMywicGF0aCI6Ii8xMTEzMTc3NS8zNDE4MDQwNzktZTE4M2I5M2QtZDZkZS00NjdlLWI0YTUtMzA5ZmRlNGZiNjYxLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzExVDIzNTgyM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWZiZjg2YmQ0Y2RhMmE1OTIyODViZGRhZDA2ODVjOTVhNjI3NjUwZTFmOWNlNmZmODQwNGE5ZjNjNmQ1NDEzMGImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.gqU8kKePQIzAMEspA7iozw5M5dXwOnYIwX09gVmZyS8)
With the sticky appearance:
![Screenshot 2024-06-21 at 10 55 13](https://private-user-images.githubusercontent.com/11131775/341804087-9b64c100-5d9a-4024-a133-3fc4c5ba319d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3NDI2MDMsIm5iZiI6MTcyMDc0MjMwMywicGF0aCI6Ii8xMTEzMTc3NS8zNDE4MDQwODctOWI2NGMxMDAtNWQ5YS00MDI0LWExMzMtM2ZjNGM1YmEzMTlkLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzExVDIzNTgyM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTI1MTM3Yzc0Y2Y5YTcwMjhjMjk1ODg2YmEzMzUxZDRlZmVjMTAyNzZkNWI2ZDMzZWE4ZTRhMDhmMzJkYzFkNDMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.TeQMK0d2YIpkUAyzkMp4HNJlSFiJE4EHaoi6MLaHs3A)
Side by side:
![cropped_normal](https://private-user-images.githubusercontent.com/11131775/341804849-c7576911-91ac-4f16-8d5f-fa945fce4264.jpg?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3NDI2MDMsIm5iZiI6MTcyMDc0MjMwMywicGF0aCI6Ii8xMTEzMTc3NS8zNDE4MDQ4NDktYzc1NzY5MTEtOTFhYy00ZjE2LThkNWYtZmE5NDVmY2U0MjY0LmpwZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzExVDIzNTgyM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWIxMWI1MjgyZGE0NWRhNGU1NDc1YWJlY2U2NzkyNjllYTlkNzMyOTVkNDVlZTkwZmM3M2MxZmFhNjUzZjBhOTUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.LPTrUR4SthBeWkB8legDE_GVCf_vfZAaSrCT3RC-LHM)
![cropped_sticky](https://private-user-images.githubusercontent.com/11131775/341804854-13e9c10d-1b22-40e1-82d0-a6374f126d4e.jpg?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3NDI2MDMsIm5iZiI6MTcyMDc0MjMwMywicGF0aCI6Ii8xMTEzMTc3NS8zNDE4MDQ4NTQtMTNlOWMxMGQtMWIyMi00MGUxLTgyZDAtYTYzNzRmMTI2ZDRlLmpwZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzExVDIzNTgyM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTRkYWJhMjY1NDc5MWRlNjlmZjE2NTdmNThlMDNiZDdkYzlkZGI4ZTg4ZDUyOWQwN2VmYjVjMDIxOGI2MDYxZmQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.BO47ZUD2aoLmgw5rqgz9WXCcQyyaWP3IJu7RsDRY848)
After experimenting a bit with the
letterSpacing
configuration, I suspect the sticky bar is rendered with a spacingterminal.integrated.letterSpacing + 1
- it always seems to be wider spacing than the rest of the terminal.VS Code version: Code - Insiders 1.91.0-insider (63d0666, 2024-06-21T10:24:54.816Z)
OS version: Darwin x64 23.5.0
Modes:
System Info
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
webgpu: enabled
A/B Experiments
The text was updated successfully, but these errors were encountered: