Respect ui.diff.tool - Githubissues

Cretezy / lazyjj

TUI for Jujutsu/jj

Apache License 2.0

34 stars 2 forks source link

Open maralorn opened 2 weeks ago

maralorn commented 2 weeks ago

Just a quick reminder that lazyjj is part of my productive daily workflow and it is awesome. <3

It would be nice if lazyjj would produce the same diff as jj.

Switching between git and the normal diff is already nice. Premium would be imo to cycle between, ui.diff.tool, default and git.

Cretezy commented 1 week ago

Great idea, will take a look at implementing this!

Cretezy commented 1 day ago

Few notes:

I agree that cycling between the options makes sense.
I think ui.diff.tool should only be included in the cycle if it's set. It should be the default.
The lazyjj.diff-format key should override the default (as it does currently), with the tool option (same as blank) available.

I am working on a refactor to the panels/design, which will make this easier to implement once finished. This feature request will come after that.