Color of diff changes when scrolling up and down about tanka

[issue-label-bot]

Issue-Label Bot is automatically applying the label kind/bug to this issue, with a confidence of 0.68. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

[cstyan]

I've had the same thing happen:

tk version v0.3.0-11-g832fedb
Linux 5.2.14-arch2-1-ARCH x86_64 GNU/Linux
alacritty 0.3.3

[codesome]

My versions:

• tk built from master at 06e745e
• Linux 5.0.0-29-generic #31~18.04.1-Ubuntu x86_64 GNU/Linux

[sh0rez]

Alright, I can reproduce this (kind of). It behaves slightly differently but similar.

Looks like this happens only in combination with less, somebody might want to figure out what less needs to be told to not produce this issue

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[cstyan]

bump, go away stale bot

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[codesome]

👀 (for the stale bot to go away)

[sh0rez]

Tbh, I have no idea how to fix this.

This issue only happens in combination with less, but I have no idea if we can tell less to not do this / if it is even possible to retain color output when scrolling up and down.

After all, colors in terminals are prefixed with these ANSI codes ... scrolling backwards means they would need to be prepended in the opposite order .. not easy, less would need to take care of this. Scrolling without a pager works fine, because it scrolls the output buffer of the terminal emulator up and down, instead of rewriting the entire screen

[cstyan]

Can we have the option of just dumping the output to stdout instead of less? I'd prefer to scroll via my own terminal or pipe to less explicitly.

[sh0rez]

Tanka tries to use your systems pager for this .. if you don't want one at all, would setting PAGER=cat work?

[cstyan]

Yes, that works via a shell function or alias. Is there any documentation anywhere of what env vars (:fire: ) tanka is making use of?

[sh0rez]

Afaict that’s the only one .. and no it’s not documented, but I will add it once I rewrite the docs

[cstyan]

For what it's worth I don't have this issue with git diff pagination

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.