Merge pull request #3248 from weaveworks/fixes-scope-linux-term-issues

Fixes monospace font overlapping in terminal+linux
This commit is contained in:
Simon
2018-07-03 10:32:43 +02:00
committed by GitHub

View File

@@ -146,6 +146,15 @@ class Terminal extends React.Component {
mountTerminal() {
Term.applyAddon(fit);
this.term = new Term({
//
// Some linux systems fail to render 'monospace' on `<canvas>` correctly:
// https://github.com/xtermjs/xterm.js/issues/1170
// `theme.fontFamilies.monospace` doesn't provide many options so we add
// some here that are very common. The alternative _might_ be to bundle Roboto-Mono
//
fontFamily: '"Roboto Mono", "Courier New", "Courier", monospace',
// `theme.fontSizes.tiny` (`"12px"`) is a string and we need an int here.
fontSize: 12,
convertEol: !this.props.pipe.get('raw'),
cursorBlink: true,
scrollback: 10000,