Skip to content
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: blame view for a diff #1308

Open
vt-alt opened this issue Dec 9, 2023 · 2 comments
Open

FR: blame view for a diff #1308

vt-alt opened this issue Dec 9, 2023 · 2 comments

Comments

@vt-alt
Copy link

vt-alt commented Dec 9, 2023

Sometimes (for example when porting) it's useful to see a blame view for a diff output. It could be cool if tig supports this mode.

Also, this could be useful in commit view -- if user will be able to toggle viewing commit diff and blamed commit diff.

@adlai
Copy link

adlai commented Jan 11, 2025

I was first going to respond that this request is illogical, because git-blame only works on individual commits; however, after further consideration of what you might have intended, I think you're referring to diffs between ranges of commits, where the blame output is nontrivial... however, it is probably better implemented first as an extension to git-blame rather than glued onto tig.

@vt-alt
Copy link
Author

vt-alt commented Jan 13, 2025

Benefit of tig is that we would conveniently scroll-and-view related commits in TUI (the way similar to tig blame). If implementing blamed diff logic is too complicated for tig (it's implemented in numerous other scripts, though), alternative would be to allow tig to view pre-geenrated blamed diff output (in a view similar to tig blame). Integrated diffing would be better than just pre-generated diff viewer because , key could work and that key is super useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants