From 979255d2987abc31c8ef5f66a900c9cedcad6ef0 Mon Sep 17 00:00:00 2001
From: Eric Liu <ericyl.us@gmail.com>
Date: Tue, 21 Nov 2023 11:45:29 -0800
Subject: [PATCH] docs: rework keyboard shortcuts to focus search bar

---
 docs/src/pages/_layout.svelte | 29 ++++++++---------------------
 1 file changed, 8 insertions(+), 21 deletions(-)

diff --git a/docs/src/pages/_layout.svelte b/docs/src/pages/_layout.svelte
index 2ac02cf2..6ab62ab8 100644
--- a/docs/src/pages/_layout.svelte
+++ b/docs/src/pages/_layout.svelte
@@ -1,8 +1,3 @@
-<script context="module">
-  // Pressing these keys will focus the search bar.
-  const FOCUS_KEYS = new Set(["/", "Metak"]);
-</script>
-
 <script>
   import {
     isActive,
@@ -47,7 +42,6 @@
   const deprecated = [];
   const new_components = [];
 
-  let keys = "";
   let value = "";
   let active = false;
   $: results = miniSearch.search(value).slice(0, 10);
@@ -72,25 +66,18 @@
 
 <svelte:body
   on:keydown="{(e) => {
-    keys += e.key;
-  }}"
-  on:keyup="{(e) => {
     if (active) return;
-
-    if (FOCUS_KEYS.has(keys)) {
-      if (
-        document.activeElement instanceof HTMLInputElement ||
-        document.activeElement instanceof HTMLTextAreaElement
-      ) {
-        // Exit early if an inputtable element is already focused.
-        return;
-      }
-
+    if (
+      document.activeElement instanceof HTMLInputElement ||
+      document.activeElement instanceof HTMLTextAreaElement
+    ) {
+      // Exit early if an inputtable element is already focused.
+      return;
+    }
+    if ((e.metaKey && e.key === 'k') || e.key === '/') {
       e.preventDefault();
       active = true;
     }
-
-    keys = '';
   }}" />
 
 <Theme persist bind:theme="{$theme}">