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

Only relevant commits for range #13

Closed
cpfaff opened this issue Jan 26, 2016 · 4 comments
Closed

Only relevant commits for range #13

cpfaff opened this issue Jan 26, 2016 · 4 comments

Comments

@cpfaff
Copy link

cpfaff commented Jan 26, 2016

Will there be a command to only show relevant commits to a range of lines that have been visual selected?

@junegunn
Copy link
Owner

Interesting suggestion, thanks. :GV command is now enabled in visual mode and in that case -L option is set which tracks down the commits for the selected lines. Needs some polishing though, but it works as expected.

junegunn added a commit that referenced this issue Jan 26, 2016
- Syntax highlighting
- More precise SHA detection
@cpfaff
Copy link
Author

cpfaff commented Jan 27, 2016

Wow that was fast thank you. Tried it and it works nicely. However I find it a bit confusing having the patches listed all in one buffer all opened to browse through. Could you implement the same behaviour as it is with just calling :GV. Having a browser like experience and that selecting a commit on the left then opens the patch on the right? That would be amazing.

@junegunn
Copy link
Owner

I felt the same way at first. But that's how git log -L works, and this plugin does not perform any post-processing on the output of the command which is why it's much faster than gitv, and I'm not going to change that. So until git adds a native option to exclude the diff content, we'll stick with the current format. Note that you can move between commits with ]] and [[ mappings. Those might help in a little.

@cpfaff
Copy link
Author

cpfaff commented Jan 27, 2016

OK. Thanks. I will try out the mappings.

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

No branches or pull requests

2 participants