Do not consider `-` part of tokens to highlight
This is a trade-off:
In most programming languages, `-` cannot be part of an identifier. On
the contrary, it is often used in operators, such as for subtraction or
as part of `->` it C++ pointer dereferencing.
In other languages such as CSS - can be part of an identifier
(e.g. 'some-css-class).
I am argueing that `-` being part of a word/token is rarer: Regexp \w
does not contain `-` for example, and if you double click on a -
separated word in the browser, it will only select the segment until the
-.
In the long term, we might make this dependent on the input language,
until then I think that not including - is better than incuding it.
Change-Id: Ic843c87db5fb75aecceb7581654ef2b1a6c1793b
diff --git a/polygerrit-ui/app/elements/diff/gr-diff-builder/token-highlight-layer.ts b/polygerrit-ui/app/elements/diff/gr-diff-builder/token-highlight-layer.ts
index c8b3901..f1ee83e 100644
--- a/polygerrit-ui/app/elements/diff/gr-diff-builder/token-highlight-layer.ts
+++ b/polygerrit-ui/app/elements/diff/gr-diff-builder/token-highlight-layer.ts
@@ -23,7 +23,7 @@
lineNumberToNumber,
} from '../gr-diff/gr-diff-utils';
-const tokenMatcher = new RegExp(/[a-zA-Z0-9_-]+/g);
+const tokenMatcher = new RegExp(/[a-zA-Z0-9_]+/g);
/** CSS class for all tokens. */
const CSS_TOKEN = 'token';
