delta icon indicating copy to clipboard operation
delta copied to clipboard

Update docs to make 3 basic configs match

Open injust opened this issue 6 months ago • 2 comments

~~Defaulting to auto light/dark detection is the smarter choice here. Besides, the commands below omit the delta.dark = true setting.~~

The basic config appears in 3 places, so make them match, to consistently de-emphasize auto-detection of colors.

injust avatar Jul 13 '25 10:07 injust

Hi @injust, I believe we initially encouraged auto-detection in the docs, and I subsequently back-pedalled a bit: https://github.com/dandavison/delta/commit/1fb6f99ec498150f552c1c8b5440152c0c2aade7 I didn't do a good job of linking that commit to an issue but I suspect it's one of the ones where the latency associated with the terminal I/O is huge on Windows.

dandavison avatar Jul 15 '25 12:07 dandavison

I see, that makes sense. In that case, since the basic config appears in 3 places, I'll change the PR to make those match.

(Also turns out the color auto-detection doesn't apply to me because I use Catppuccin, and what I actually need is https://github.com/dandavison/delta/issues/1678.)

injust avatar Jul 15 '25 13:07 injust