colorediffs icon indicating copy to clipboard operation
colorediffs copied to clipboard

Feature request: add quick links if diff has more than 1 file change

Open aik opened this issue 8 years ago • 3 comments

$subj

aik avatar Nov 15 '16 02:11 aik

Hi, and thanks! Am I correct to suppose that by adding “quick links”, you mean adding a list of modified files (with the links) similar to a ToC at the beginning of the diff for quick jumps? Or did I miss the point?

Qeole avatar Nov 16 '16 19:11 Qeole

No, you got it right :) A patch may change 25 files, and I only care about few - so I need a way to jump quickly to these files. I could think of a drop-down list of files where all the addon controls are ooooor replace file names in the index with clickable links (I do not know if it is doable though). By "index" I mean this:

include/sysemu/numa.h | 2 +- include/sysemu/sysemu.h | 7 ------- numa.c | 19 ++++++++++++------- vl.c | 5 ----- 4 files changed, 13 insertions(+), 20 deletions(-)

aik avatar Nov 17 '16 02:11 aik

Ok, understood. I like (more) the second idea, adding links to the names in the index. I became maintainer of this add-on some months ago, but I don't know enough of the code yet to tell how easy that would be to implement.

Also, adding features here is not at the top of my todo-list, so do not expect me to do it really soon, I'll see when I have some spare time (or if anyone else wants to give it a try, don't hesitate).

Meanwhile, thanks for the idea and for using the add-on!

Qeole avatar Nov 17 '16 18:11 Qeole