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 a Diff View #1284

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Add a Diff View #1284

wants to merge 9 commits into from

Conversation

joachimm
Copy link
Member

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.

@joachimm joachimm changed the title A Diff View for TextMate. Add a Diff View Jan 26, 2015
@enure
Copy link

enure commented Mar 7, 2016

The screencast looks nice. It might be nice also to have an option to just mark lines that have changed in the gutter.

@koenpunt
Copy link
Contributor

koenpunt commented Mar 9, 2016

@enure that is already possible with the git bundle

@enure
Copy link

enure commented Mar 10, 2016

@koenpunt Thanks for the tip. By the way, it's actually the SCM Diff Gutter plugin.

@koenpunt
Copy link
Contributor

Ah, you're right, of course :)

@attilagyorffy
Copy link

This is really cool :) Any chance for a merge at some point?

@sorbits
Copy link
Member

sorbits commented Jan 26, 2017

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.

@joachimm
Copy link
Member Author

joachimm commented Mar 5, 2017

@sorbits I have a rebased branch I could push if there is interest.

@dantiel
Copy link

dantiel commented Oct 17, 2017

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants