search

tpope / vim-fugitive

fugitive.vim: A Git wrapper so awesome, it should be illegal
https://www.vim.org/scripts/script.php?script_id=2975
19.82k stars 1k forks source link

In `:Git blame`, use color as a proxy for commit age #2324

Closed jez closed 1 month ago

jez commented 1 month ago

In GitHub's blame view, there's a little legend showing that older commits are accented with a light yellow color, and the lightness/brightness of the shade of yellow drops all the way to pitch black for very young commits. See it in action here:

https://github.com/tpope/vim-fugitive/blame/0444df68cd1cdabc7453d6bd84099458327e5513/autoload/fugitive.vim

Screenshot 2024-07-31 at 2 38 39 PM

It would be super cool if vim-fugitive could do the same. Happy to work on this, but I didn't want to start working on it if the feature had been considered and explicitly been rejected for consideration. I searched the issue list and didn't see anything specifically mentioning this but it's possible I just didn't search thoroughly enough.

tpope commented 1 month ago

I not just considered but implemented it, but I never merged it because I wasn't happy with how it looked. See https://github.com/tpope/vim-fugitive/issues/901#issuecomment-509017914.