Setting line numbers in `less` messes up ruler

[alok]

See so-fancy/diff-so-fancy#271 for an explanation, since it's the same issue.

[dandavison]

If we added a command line option allowing the user to specify how many columns they want delta to use when filling right to the edge of the terminal, do you think that would solve your problem? I haven't fully thought it through, but I did skim your so-fancy/diff-so-fancy#271. Are you seeing the same problem with the background colors extending onto the next line, or just the ruler?

[alok]

The background colors also run.

I think it would fix it. I'd be willing to test it out, since I use Rust and can install a branch from git to try it.

[dandavison]

OK great, it's on the list. (This would also make a good first issue for anyone who wants a fairly doable Rust task.)

[dandavison]

Hi @alok, this should be fixed in master: the background colors now adapt dynamically to the terminal width (see 5b26209) and the horizontal decoration lines now respect the --width option.

[lainisourgod]

I'm still having this.

Have bat as $PAGER and $DELTA_PAGER

when I do git diff | delta | bat it's ok

when I do just git diff with delta set as pager, line is shifted by this line number column

[dandavison]

Hi @meandmymind, you can solve this by setting delta width to a negative value, which means auto-detected width minus $value, e.g.

[delta]
    width = -9

I'm sorry this wasn't documented. The reason was slightly ridiculous -- somehow the wrong sort of comments had got onto two lines of the cli.rs modules, so that the relevant two lines of documentation didn't appear in the --help output... So thanks for alerting me to that. Fixed in 5ea8dfc and --help output is in README.

[lainisourgod]

hi @dandavison that's cool!

but why is it happening? I don't have any less and bat configuration in my system, so "minusing 7" should be default behaviour?

[dandavison]

It's because you've got DELTA_PAGER=bat. I didn't say anything because I thought it was deliberate, but tbh I did think it was a strange choice :) If you have PAGER=bat then you should set DELTA_PAGER to something like less -R.

[lainisourgod]

@dandavison ok it means I don't understand this paging stuff but changing DELTA_PAGER to less -R worked :))) thanks

[dandavison]

@meandmymind Would it help if I added something like this to the README? (which maybe should become a user manual one day) I'm not implying that you need all these definitions explained! Trying to make it clear to as many people as possible.

Glossary

• terminal emulator: a graphical application that mimics an old-fashioned text terminal. You run a shell in a terminal emulator (e.g. iTerm2, Terminal.app, Windows Terminal, Gnome Terminal, Konsole, Alacritty, are terminal emulators)
• shell: an application that displays a prompt, at which you can run command line applications (e.g. bash is an example shell)
• command-line application: an application that you run by typing its name at a shell prompt with some arguments. It takes in text, and outputs text. Sometimes the text can have special color codes in it to make the terminal emulator print the text in color.
• pager: a command line application for displaying any sort of text one screenful at a time (e.g. less is a pager)
• less: the standard pager that is used in linux and unix environments. less is suitable for paging any sort of input.
• bat: a command line program for displaying source code with syntax highlighting. bat sends its output to less for paging
• delta: a command line program for displaying specialist file formats such as git, diff, grep, and git blame output, with code syntax highlighting. delta also sends its output to less for paging
• environment variable: a variable that you can set in your shell session
• PAGER: an environment variable that can be used to specify the way you want programs to call less, when those programs need to do paging. For example you could do export PAGER="less -R" to set this variable: -R tells less to display colors when its input contains color codes.
• BAT_PAGER: an environment variable that you can set to tell bat to call less differently from the way other programs call less
• DELTA_PAGER: an environment variable that you can set to tell delta to call less differently from the way other programs call less
• LESS: an environment variable that you can set to specify the default arguments less should be called with. (Yes, at this point things have got a bit convoluted! You probably want to ignore this one.)

Tips

• You don't need to set any environment variables unless doing so fixes a problem for you!
• Type env to print out a list of the environment variables you have set, to check that PAGER, BAT_PAGER, DELTA, LESS are not set or, if they are set, that they have sensible values.