Skip to content
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

Terminal font squished when changing monitors with varying DPIs #34320

Closed
knownasilya opened this issue Sep 13, 2017 · 4 comments
Closed

Terminal font squished when changing monitors with varying DPIs #34320

knownasilya opened this issue Sep 13, 2017 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Milestone

Comments

@knownasilya
Copy link

  • VSCode Version: Code - Insiders 1.17.0-insider (06529db, 2017-09-13T05:32:25.862Z)
  • OS Version: Darwin x64 16.7.0
  • Extensions:
Extension Author (truncated) Version
dart-code Dan 2.4.0
beautify Hoo 1.1.1
material-icon-theme PKi 2.2.1
vscode-docker Pet 0.0.16
lorem-ipsum Tyr 1.0.0
quokka-vscode Wal 1.0.63
Bookmarks ale 0.16.0
project-manager ale 0.21.1
workbench and 0.1.6
typings-installer ben 0.2.0
npm-intellisense chr 1.3.0
vsc-ember-frost cie 0.0.2
vscode-eslint dba 1.3.1
githistory don 0.2.3
tslint eg2 1.0.4
vscode-npm-script eg2 0.2.1
vscode-ember emb 0.1.0
csharp ms- 1.12.1
debugger-for-chrome msj 3.2.1
stylelint shi 0.29.0
ayu tea 0.6.0
vim vsc 0.10.0

Steps to Reproduce:

  1. Open terminal in a non-retina monitor
  2. Drag vscode to retina monitor
  3. Click expand on the terminal panel
  4. Terminal font is super tiny and impossible to read
  5. Move back to non-retina monitor and collapse, it's back to normal

Also the fonts are fuzzy on both monitors, only in the terminal.

terminal

@vscodebot vscodebot bot added the insiders label Sep 13, 2017
@vscodebot vscodebot bot added the terminal General terminal issues that don't fall under another label label Sep 13, 2017
@Tyriar Tyriar added this to the September 2017 milestone Sep 13, 2017
@Tyriar Tyriar added the bug Issue identified by VS Code Team member as probable bug label Sep 13, 2017
@Tyriar Tyriar modified the milestones: September 2017, October 2017 Sep 26, 2017
@chrmarti chrmarti removed the insiders label Sep 27, 2017
@Tyriar
Copy link
Member

Tyriar commented Oct 2, 2017

May need to use the 'display-metrics-changed' event from Electron to detect DPI changes when dragging the window, I was expecting window.onresize to fire but it doesn't.

@Tyriar
Copy link
Member

Tyriar commented Oct 2, 2017

Could also include the target devicePixelRatio with the CharAtlas and verify that before drawing.

@Tyriar
Copy link
Member

Tyriar commented Nov 8, 2017

Created the upstream issue xtermjs/xterm.js#1118

@abdel-ships-it
Copy link

Very frustrating bug, resizing the terminal seems to fix it.

@Tyriar Tyriar removed this from the Backlog milestone Dec 28, 2017
@Tyriar Tyriar modified the milestones: On Deck, December 2017/January 2018 Dec 28, 2017
@Tyriar Tyriar closed this as completed in 2cc5e38 Dec 29, 2017
@Tyriar Tyriar added the upstream Issue identified as 'upstream' component related (exists outside of VS Code) label Dec 29, 2017
@aeschli aeschli added the verified Verification succeeded label Feb 1, 2018
@Tyriar Tyriar changed the title Terminal font squished in retina display Terminal font squished when changing monitors with varying DPIs Feb 3, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Feb 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants