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

Add "Launch Diff Tool" option to DAG viewer #468

Closed
nocnokneo opened this issue Jun 30, 2015 · 7 comments
Closed

Add "Launch Diff Tool" option to DAG viewer #468

nocnokneo opened this issue Jun 30, 2015 · 7 comments

Comments

@nocnokneo
Copy link

Just like the main window allows for viewing changes to an individual file in an external diff viewer, it would be nice to be able to view changesets in an external diff viewer (i.e. using git difftool -d ... mode). There would be two ways of using this feature:

  • Select a single commit (either in the Log or the Graph view), then right click to get a "Launch Diff Tool" context menu option (or just use Ctrl-D shortcut). This would show the differences between the selected commit and its parent (git difftool -d xxxxxxx^..xxxxxxx)
  • Select two commits by holding Ctrl and selecting two commits (either in the Log or the Graph view), then right click to get a "Launch Diff Tool" context menu option (or just use Ctrl-D shortcut). This would show the differences between the two selected commits (git difftool -d xxxxxxx..yyyyyyy)

Related side note: selecting two commits in the Log view is currently broken for me - holding Ctrl while clicking multiple commits behaves as if I where holding Shift.

@davvid
Copy link
Member

davvid commented Jul 1, 2015

Sounds like good idea. Right now the log view should work for diffing commits by clicking one, and then, without trying to select it, just right-click a different commit. Like gitk, it should have "Diff selected -> this", and vice versa in the context menu.

davvid added a commit that referenced this issue Nov 7, 2015
Related-to: #468
Suggested-by: Taylor Braun-Jones <taylor@braun-jones.org>
Signed-off-by: David Aguilar <davvid@gmail.com>
AndiDog pushed a commit to AndiDog/git-cola that referenced this issue Nov 7, 2015
Related-to: git-cola#468
Suggested-by: Taylor Braun-Jones <taylor@braun-jones.org>
Signed-off-by: David Aguilar <davvid@gmail.com>
@feinstaub
Copy link

What about adding a new button "Show in external diff tool" to the embedded diff viewer? So by default one would see the diff in the embedded viewer and when this is not enough the external one can be launched.

@davvid
Copy link
Member

davvid commented Feb 8, 2016

That sounds like it'd be pretty useful.

@nocnokneo
Copy link
Author

I agree - sounds like a good idea. It goes against git-cola's current UI pattern a bit (minimal buttons; most operations accessed by context menu or keyboard shortcut) but I'd still say it makes sense here.

@davvid davvid closed this as completed in bc30de2 Sep 9, 2016
@nocnokneo
Copy link
Author

Cool, thanks. Although I was hoping to get a directory diff (git difftool -d ...) somehow. Could we add a "Compare All" button to the "git-cola diff" dialog?

@davvid
Copy link
Member

davvid commented Sep 10, 2016

I'll take your "Compare All" button and raise you a Ctrl+Enter hotkey (for that dialog) and a standalone Ctrl+Shift+D hotkey for launching dir-diff via git-dag.

davvid added a commit that referenced this issue Sep 10, 2016
Reuse the left_take_parent logic to add a helper that adds the '^!' diff
suffix for easily seeing the contents of a specific commit.

Related-to: #468
Signed-off-by: David Aguilar <davvid@gmail.com>
davvid added a commit that referenced this issue Sep 10, 2016
Add a Ctrl+Enter hotkey and "Compare All" button for launching directory
diffs using the difftool dialog.

Related-to: #468
Suggested-by: Taylor Braun-Jones <taylor@braun-jones.org>
Signed-off-by: David Aguilar <davvid@gmail.com>
davvid added a commit that referenced this issue Sep 10, 2016
Make `git difftool --dir-diff` more accessible by adding a Ctrl+Enter
hotkey for launching directory diffs.

Related-to: #468
Suggested-by: Taylor Braun-Jones <taylor@braun-jones.org>
Signed-off-by: David Aguilar <davvid@gmail.com>
@nocnokneo
Copy link
Author

nocnokneo commented Sep 10, 2016 via email

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

3 participants