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

Fixes monospace font overlapping in terminal+linux #3248

Merged
merged 1 commit into from
Jul 3, 2018

Conversation

foot
Copy link
Contributor

@foot foot commented Jul 2, 2018

Fixes #3245

I believe this fixes the linked issue but I haven't had a chance to test under linux yet..

@rade
Copy link
Member

rade commented Jul 2, 2018

I have just run this branch and my terminal is now not showing anything except cursor movement as I type; no characters are rendered at all.

@rade
Copy link
Member

rade commented Jul 2, 2018

Works fine in Chrome though. Not FF.

@foot
Copy link
Contributor Author

foot commented Jul 2, 2018

@rade thanks for testing! I will try something else...

@foot
Copy link
Contributor Author

foot commented Jul 2, 2018

I have just run this branch and my terminal is now not showing anything except cursor movement as I type; no characters are rendered at all.

I got a linux env of virtualbox+ubuntu18.04+firefox59.0.2 and I can repro and fix. I've tried installing and removing roboto in the linux vm. It might be falling back to a strange default font on your machine now.

I've popped in another font there as a bit of a hail mary, @rade could you please try testing this updated PR locally?

@rade
Copy link
Member

rade commented Jul 2, 2018

Works fine now, though for some reason the first time I opened the terminal it was still blank.

- Also fixes actually using Roboto-Mono, xterm.js now only accepts
  fontFamily customization via the JS api. (uses `<canvas`).
- There is an issue (xtermjs/xterm.js#1170)
  if linux has to fall back to 'monospace' where things fail to render
  correctly, so provide a couple of common options so this rarely
  happens.
- If we distribute robo-mono in the webpack bundle that might also work.
@foot foot force-pushed the fixes-scope-linux-term-issues branch from 1dbe145 to 7a04740 Compare July 2, 2018 16:01
@foot foot requested a review from jpellizzari July 2, 2018 16:03
@foot
Copy link
Contributor Author

foot commented Jul 2, 2018

@jpellizzari not sure if you're on linux, but @rade and I verified that it works there. Code looks okay?

Copy link
Contributor

@jpellizzari jpellizzari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@foot foot merged commit 3656513 into master Jul 3, 2018
@foot foot deleted the fixes-scope-linux-term-issues branch July 3, 2018 08:32
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.

lines overlapping in terminals
3 participants