extern/hishtory
1
0
Fork 0
mirror of https://github.com/ddworken/hishtory.git synced 2024-11-29 11:44:53 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix broken custom color scheme support as reported in #134

Browse Source
This commit is contained in:
David Dworken 2023-12-28 19:24:51 -08:00
parent 9ac3df1bbb
commit 30f94c1c78
No known key found for this signature in database
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
client/tui/tui.go
View File

@ -812,6 +812,24 @@ func TuiQuery(ctx context.Context, initialQuery string) error {
// test environments behave the same (by default, github actions
// ubuntu and macos have different termenv support).
lipgloss.SetColorProfile(termenv.ANSI)
} else {
// When the shell launches control-R it isn't hooked up to the main TTY,
// which means that termenv isn't able to accurately detect color support
// in the current terminal. This means we have to guess the right option,
// where we risk either:
// * Choosing too high of a color support, and breaking hishtory colors
// in certain terminals
// * Choosing too low of a color support, and ending up with truncating
// customized colors
//
// This is a tough situation with no right answer (as far as I can tell).
// The default terminal app on MacOS only supports termenv.ANSI256 (8 bit
// colors), which means we likely shouldn't default to TrueColor. From
// my own digging, I can't find any modern terminals that don't support
// termenv.ANSI256, so it seems like a reasonable default here.
//
// TODO: In the long term, we may want to make this configurable.
lipgloss.SetColorProfile(termenv.ANSI256)
}
p := tea.NewProgram(initialModel(ctx, initialQuery), tea.WithOutput(os.Stderr))
// Async: Get the initial set of rows