Improve tag scanning behaviour (#1189)

* Update include/exclude selectors for Display._definitionTextScanner

* Fix missing condition

2 files changed, 7 insertions, 7 deletions

diff --git a/ext/mixed/js/display.js b/ext/mixed/js/display.js
index c0d84dce..3817559d 100644
--- a/ext/mixed/js/display.js
+++ b/ext/mixed/js/display.js
@@ -845,6 +845,7 @@ class Display extends EventDispatcher {
     _showTagNotification(content) {
         if (this._tagNotification === null) {
             const node = this._displayGenerator.createEmptyFooterNotification();
+            node.classList.add('click-scannable');
             this._tagNotification = new DisplayNotification(this._tagNotificationContainer, node);
         }
 
@@ -1792,8 +1793,8 @@ class Display extends EventDispatcher {
                 searchOnClick: true,
                 searchOnClickOnly: true
             });
-            const excludeSelectors = ['.scan-disable', '.scan-disable *'];
-            this._definitionTextScanner.excludeSelector = excludeSelectors.join(',');
+            this._definitionTextScanner.includeSelector = '.click-scannable,.click-scannable *';
+            this._definitionTextScanner.excludeSelector = '.scan-disable,.scan-disable *';
             this._definitionTextScanner.prepare();
             this._definitionTextScanner.on('searched', this._onDefinitionTextScannerSearched.bind(this));
         }
@@ -1825,9 +1826,6 @@ class Display extends EventDispatcher {
             preventMiddleMouse: false
         });
 
-        const includeSelector = '.term-glossary-item,.term-glossary-item *,.tag,.tag *';
-        this._definitionTextScanner.includeSelector = includeSelector;
-
         this._definitionTextScanner.setEnabled(true);
     }
 
diff --git a/ext/mixed/js/text-scanner.js b/ext/mixed/js/text-scanner.js
index f0903370..daea6a6c 100644
--- a/ext/mixed/js/text-scanner.js
+++ b/ext/mixed/js/text-scanner.js
@@ -198,8 +198,10 @@ class TextScanner extends EventDispatcher {
 
         clonedTextSource.setEndOffset(length, layoutAwareScan);
 
-        if (this._excludeSelector !== null) {
-            this._constrainTextSource(clonedTextSource, this._includeSelector, this._excludeSelector, layoutAwareScan);
+        const includeSelector = this._includeSelector;
+        const excludeSelector = this._excludeSelector;
+        if (includeSelector !== null || excludeSelector !== null) {
+            this._constrainTextSource(clonedTextSource, includeSelector, excludeSelector, layoutAwareScan);
         }
 
         return clonedTextSource.text();