-
Notifications
You must be signed in to change notification settings - Fork 405
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
Side by side viewer #86
Comments
Can you expand on what this feature would be? I don't think it's been discussed yet. So far, delta is based around parsing default git output and unified diff output (which is very similar). |
Lots of IDEs and GUI diff tools show differences side-by-side, instead of on different lines in a unified output like Do you think you would ever consider something like that? It may be out of the scope of this project, which I would completely understand. But I figured I would ask 😄 |
Hi @HarrisonMc555, thanks! OK, so, delta doesn't have access to the original files being compared: what delta is is a tool for applying syntax highlighting to the output of diff programs (such as So delta itself would not generate the side-by-side view (it can't, it doesn't have access to the original files). Delta could however process the output of a program such as OK, here's a question. I wonder whether, rather than using delta, there's a solution that uses
|
Thanks for your response! I'll have to think about it for a bit. Thanks for your help! |
No problem, I'll close this for now, but feel free to add any further thoughts. |
@dandavison I strongly agree @HarrisonMc555 's idea. This feature would be so awesome with this amazing tool. |
Thanks @Dentrax and @HarrisonMc555. I think I said something wrong above. It's possible for delta to construct a side-by-side view without access to the original files. I agree that this is a feature that would very much make sense. |
+1 👍 @Dentrax Thanks for sharing, quite interesting. |
@dandavison would you like any help with this issue? not making any promises but I'd be interested in looking into it |
@mscottnelson thanks, that would be great! So I'll give you a bit of time to take a look at what needs to be done -- I won't do anything in this area without pinging you on this ticket first. I'm currently working on getting the next release out (0.2.0). But let's not try to include this feature in that release -- we can release this when it's ready. I'm excited about this feature, but there's also no shortage of other delta improvements for me to work on, so very happy for others to take features like this. I believe that the recent changes in master have put us in a decent position to implement this. For example, line numbers have been added to delta by @clnoll in #190 (and open PR #219), and they have a left and right column of numbers that should fit naturally into the left and right split in side-by-side view. |
@mscottnelson are you still thinking of working on this or should we consider it open again? |
@mscottnelson I'm |
@dandavison Great! sorry for the poor communication. Thanks for being so awesome/proactive! |
Thank you so much for such a great job! @dandavison |
Side-by-side diff view has been released (delta 0.3.0). It has a section in the README, but basically you use [delta]
side-by-side = true or the equivalent command-line flags It can be combined with any other options, such as cc @Amorymeltzer, @Avi-D-coder, @Kr1ss-XD, @marcoieni, @RobbieClarken, @Valloric, @crash-g, @jeffkaufman, @jwatt, @kenchan0130, @kyohsuke, @lightspeedbriefs, @madnight, @maricn, @mosic, @mscottnelson, @navarroaxel, @nemonweb, @panchalkalpesh, @softoika, @wataash, @zdenekkostal (Thanks @Dentrax!) |
This feature really rocks! 👏 |
Are there any plans to support side-by-side viewing?
The text was updated successfully, but these errors were encountered: