-
Notifications
You must be signed in to change notification settings - Fork 750
Open
Labels
polish🪒🐃Make existing features more convenient and more consistentMake existing features more convenient and more consistent
Description
Is your feature request related to a problem? Please describe.
The possibilities to configure how the diffs are displayed are great, I just have one annoyance left I couldn't get over.
- Having all tokens underlined is for me very hard to read and also is easily confusable with underscores being added or removed, especially in whitespace-only tokens, so I disabled underlining using
colors."diff token".underline
.
Withyou don't really know whether it is
foo baz
tofoo barbaz
or whether it isfoo __baz
tofoo barbaz
or maybefoo _ baz
tofoo barbaz
orfoo _baz
tofoo barbaz
. - This greatly improves the diff readability for me, except for the case where only whitespace changed in a token as the token is no longer identifiable at all, so I reversed the colors using
colors."diff token".reverse
. - This makes the whitespace-only tokens identifiable again, but for all other tokens decreases the readability again in a similar manner the underlining did, just that now you cannot confuse the underlining with an underscore.
Describe the solution you'd like
It would be amazing if I could configure reverse
to false
, underline
to false
, and additionally a setting that for leading and trailing whitespace in a diff token (and thereby also whitespace-only tokens) reverse
is true
, so that readability of other tokens is not hurt while keeping whitespace changes identifiable like this:

Metadata
Metadata
Assignees
Labels
polish🪒🐃Make existing features more convenient and more consistentMake existing features more convenient and more consistent