-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 a Diff View #1284
base: master
Are you sure you want to change the base?
Add a Diff View #1284
Conversation
The screencast looks nice. It might be nice also to have an option to just mark lines that have changed in the gutter. |
@enure that is already possible with the git bundle |
@koenpunt Thanks for the tip. By the way, it's actually the SCM Diff Gutter plugin. |
Ah, you're right, of course :) |
4234188
to
09534b7
Compare
This is really cool :) Any chance for a merge at some point? |
There has been too many changes to master for this to be merged. I have kept the PR open as I think some sort of native diff view would make sense, but not many specific thoughts ATM. |
@sorbits I have a rebased branch I could push if there is interest. |
its indeed nice to have this as an inbuilt feature , but for me a refined kaleidoscope integration would do as well, a final solution should work in both ways |
093e8eb
to
d2979e2
Compare
Some facts:
Enabled by menu item: View -> Diff
Uses the patience-diff algorithm, enhanced for live editing.
Unified view only for now.
Compare against Git HEAD only for now.
New appended ”layout” scopes: diff.removed.line and diff.added.line .
Stick this in your theme file to get started:
{ name = 'Diff Removed';
scope = 'diff.removed';
settings = { background = '#FFC0CB'; };
},
{ name = 'Diff Added';
scope = 'diff.added';
settings = { background = '#90EE90'; };
},
Screencast: http://www.screencast.com/t/ZQyRQPIBvu Missing +/- though.