From 52d8d658a2bbf0dea8fecebe2db7f80934353c91 Mon Sep 17 00:00:00 2001
From: toasted-nutbread <toasted-nutbread@users.noreply.github.com>
Date: Sat, 29 Feb 2020 10:00:28 -0500
Subject: [PATCH] Fix empty keys definitions and add Shift/Alt checks

---
 ext/bg/js/search.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/ext/bg/js/search.js b/ext/bg/js/search.js
index e9b1918b..0a7a5fe1 100644
--- a/ext/bg/js/search.js
+++ b/ext/bg/js/search.js
@@ -49,10 +49,10 @@ class DisplaySearch extends Display {
             ])],
             ['Control', new Set(['C', 'A', 'Z', 'Y', 'X', 'F', 'G'])],
             ['Meta', new Set(['C', 'A', 'Z', 'Y', 'X', 'F', 'G'])],
-            ['OS', []],
-            ['Alt', []],
-            ['AltGraph', []],
-            ['Shift', []]
+            ['OS', new Set()],
+            ['Alt', new Set()],
+            ['AltGraph', new Set()],
+            ['Shift', new Set()]
         ]);
 
         this._runtimeMessageHandlers = new Map([
@@ -175,6 +175,8 @@ class DisplaySearch extends Display {
         const activeModifierMap = new Map([
             ['Control', e.ctrlKey],
             ['Meta', e.metaKey],
+            ['Shift', e.shiftKey],
+            ['Alt', e.altKey],
             ['ANY_MOD', true]
         ]);