From b250644ea880f2d09eb7273644364fae2f18ce17 Mon Sep 17 00:00:00 2001 From: Jesse Duffield Date: Thu, 1 Jun 2023 20:12:20 +1000 Subject: [PATCH] Stop hiding underscores for VSCode VSCode had an issue in their terminal where underscores were printed all over the place. That has now been fixed. See https://github.com/jesseduffield/lazygit/issues/2294 and https://github.com/xtermjs/xterm.js/issues/4238 --- pkg/gui/style/text_style.go | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/pkg/gui/style/text_style.go b/pkg/gui/style/text_style.go index 3eee68eff..f0a1da0e6 100644 --- a/pkg/gui/style/text_style.go +++ b/pkg/gui/style/text_style.go @@ -1,8 +1,6 @@ package style import ( - "os" - "github.com/gookit/color" ) @@ -27,23 +25,6 @@ import ( // a string, we derive it when a new TextStyle is created and store it in the // `style` field. -// See https://github.com/xtermjs/xterm.js/issues/4238 -// VSCode is soon to fix this in an upcoming update. -// Once that's done, we can scrap the HIDE_UNDERSCORES variable -var ( - underscoreEnvChecked bool - hideUnderscores bool -) - -func hideUnderScores() bool { - if !underscoreEnvChecked { - hideUnderscores = os.Getenv("TERM_PROGRAM") == "vscode" - underscoreEnvChecked = true - } - - return hideUnderscores -} - type TextStyle struct { fg *Color bg *Color @@ -83,10 +64,6 @@ func (b TextStyle) SetBold() TextStyle { } func (b TextStyle) SetUnderline() TextStyle { - if hideUnderScores() { - return b - } - b.decoration.SetUnderline() b.Style = b.deriveStyle() return b