New config option: trailing_spaces_scope_ignore

Defaults to ignoring find-in-files, diff and build output views.

Also, remove mentions of the inferior `trailing_spaces_syntax_ignore`
option in the README (still considered in the code for backward compat)

Author: smanolloff

README.md

@@ -276,16 +276,22 @@ a binding for the toggling command. When "On-demand Matching" is on and some
 trailing spaces are highlighted, added ones will obviously not be. Toggling
 highlight off and on will refresh them.
 
-### Ignore Syntax
+### Ignore Scope
 
-*Default: []*
+*Default: ["text.find-in-files", "source.build_output", "source.diff"]*
 
-With this option you can ignore specific files/views based on the syntax used.
-An item has to match a case-sensitive substring of the syntax used in the view:
+With this option you can ignore lines being highlighted based on the scope of
+their trailing region.
+
+If at least one scope in the configured list matches a scope in the trailing
+region of the line, it won't be highlighted.
+
+By default, the scope under the mouse cursor is shown by pressing
+`Option+Command+P` (OS X) or `Ctrl+Alt+Shift+P` (Windows, Linux)
 
 ``` js
-// Views with a syntax that contains "Diff" are ignored
-{ "trailing_spaces_syntax_ignore": ["Diff"]}
+// Trailing spaces for find results, build output and markdown are ignored
+{ "trailing_spaces_scope_ignore": ["text.find-in-files", "source.build_output", "text.html.markdown"] }
 ```
 
 ### For power-users only!

trailing_spaces.py

@@ -67,15 +67,6 @@ def persist_settings():
     sublime.save_settings(ts_settings_filename)
 
 
-# Private: Determine if the view is a "Find results" view.
-#
-# view - the view, you know
-#
-# Returns True or False.
-def is_find_results(view):
-    return view.settings().get('syntax') and "Find Results" in view.settings().get('syntax')
-
-
 # Private: Get the regions matching trailing spaces.
 #
 # As the core regexp matches lines, the regions are, well, "per lines".
@@ -93,17 +84,23 @@ def find_trailing_spaces(view):
     no_empty_lines_regexp = "(?<=\\S)%s$" % regexp
 
     offending_lines = view.find_all(regexp if include_empty_lines else no_empty_lines_regexp)
+    ignored_scopes = ",".join(ts_settings.get("trailing_spaces_scope_ignore", []))
+    filtered_lines = []
+    for region in offending_lines:
+        if ignored_scopes and view.match_selector(region.begin(), ignored_scopes):
+            continue
+        filtered_lines.append(region)
 
     sel = view.sel()
     line = len(sel) and view.line(sel[0].b)
 
     if include_current_line or not line:
-        return [offending_lines, offending_lines]
+        return [filtered_lines, filtered_lines]
     else:
         current_offender = view.find(regexp if include_empty_lines else no_empty_lines_regexp, line.a)
         removal = False if current_offender is None else line.intersects(current_offender)
-        highlightable = [i for i in offending_lines if i != current_offender] if removal else offending_lines
-        return [offending_lines, highlightable]
+        highlightable = [i for i in filtered_lines if i != current_offender] if removal else filtered_lines
+        return [filtered_lines, highlightable]
 
 
 # Private: Find the freaking trailing spaces in the view and flags them as such!
@@ -128,10 +125,9 @@ def match_trailing_spaces(view):
     if max_size_exceeded(view):
         return
 
-    if not is_find_results(view):
-        (matched, highlightable) = find_trailing_spaces(view)
-        add_trailing_spaces_regions(view, matched)
-        highlight_trailing_spaces_regions(view, highlightable)
+    (matched, highlightable) = find_trailing_spaces(view)
+    add_trailing_spaces_regions(view, matched)
+    highlight_trailing_spaces_regions(view, highlightable)
 
 
 # Private: Checks if the view should be ignored.

trailing_spaces.sublime-settings

@@ -24,6 +24,10 @@
     // Set to false to ignore trailing spaces on the edited line.
     "trailing_spaces_include_current_line" : true,
 
+    // By default, any lines in the Find Results Build output and Diff views are ignored
+    // Add scopes to this list if you need to ignore them.
+    "trailing_spaces_scope_ignore": ["text.find-in-files", "source.build_output", "source.diff"],
+
     // By default, trailing spaces are deleted within the whole document.
     // Set to true to affect only the lines you edited since last save.
     // Trailing spaces will still be searched for and highlighted in the whole