-
Notifications
You must be signed in to change notification settings - Fork 350
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FR: Improve "diff" visibility by adding marker before line number #5140
Comments
Just fyi, you could set |
Thanks. That suggestion improves the detectability of changes, but at the same time makes it harder to compare them once detected. |
or make context line numbers dimmed. Naming aside, maybe we can add "context"/"changed" label to line numbers (or to the whole line.) That's the easiest fix. |
Now that you suggested it, I think dimming the context line numbers would be the best option of all. |
The default might have to be underline/bold changed line numbers for compatibility reason, but user can set whatever color/style if line numbers and contents are labeled as "context"/"changed". |
Is your feature request related to a problem? Please describe.
The current diff is mostly good, however, when there are small additions on a line those changes are hard to spot.
This happens because unlike red color green color doesn't standout.
Describe the solution you'd like
Add a marker on the lines that contain changes at the very beginning of the line, i.e. before line number.
Describe alternatives you've considered
An alternative would be to highlight the whole line with a background, however, I believe it might not be supported on all terminals
Additional context
I am aware of the existence of
--git
switch that with an external pager.However, the default diff seems to be good enough except for this issue and I'd avoid using external tools whenever possible.
The text was updated successfully, but these errors were encountered: