miketheman / language-diff

ARCHIVED - Atom editor itself is archived
https://atom.io/packages/language-diff
23 stars 12 forks source link

Feature Request: Better support for recursive diffs #5

Closed peterstory closed 8 years ago

peterstory commented 8 years ago

If you do a recursive diff to compare two directories, the output includes lines like:

Only in DIR_NAME: FILENAME

It would be great if the syntax could be modified to emphasize these lines in some way. Without emphasis, it's easy to overlook them, because the colored lines stand out much more. Perhaps bolding them would work well?

miketheman commented 8 years ago

Can you provide such an example in more complete form, so there's something to test it against?

peterstory commented 8 years ago

old and new versions.zip Sure thing!

Here is the output of running diff -ru old_version new_version:

diff -ru old_version/file_a.txt new_version/file_a.txt
--- old_version/file_a.txt  2016-09-30 10:55:30.000000000 -0400
+++ new_version/file_a.txt  2016-09-30 10:56:13.000000000 -0400
@@ -1,2 +1,4 @@
 This is a very important file.
 It contains a lot of useful data.
+
+No, this file doesn't contain anything useful at all! 
Only in new_version: file_b.txt
miketheman commented 8 years ago

@peterstory Thanks!

I've got a simple fix ready, where it uses the theme's colors adds bold to it and uses the function syntax highlighter. Here's what it looks like:

Solarized Dark

screenshot 2016-10-01 07 15 35

One Light

screenshot 2016-10-01 07 15 49

Atom Light

screenshot 2016-10-01 07 15 59

Let me know if you have any other thoughts before I release this in a few days.

peterstory commented 8 years ago

That looks really great! Thanks for making that improvement!

miketheman commented 8 years ago

Released. Enjoy!