fix: disable some hot keys when disabled in settings

Author: alex-key

app/components/Compare/PackageSelector.vue

@@ -75,6 +75,8 @@ const resultIndexOffset = computed(() => (showNoDependencyOption.value ? 1 : 0))
 
 const numberFormatter = useNumberFormatter()
 
+const keyboardShortcuts = useKeyboardShortcuts()
+
 function addPackage(name: string) {
   if (packages.value.length >= maxPackages.value) return
   if (packages.value.includes(name)) return
@@ -98,6 +100,10 @@ function removePackage(name: string) {
 }
 
 function handleKeydown(e: KeyboardEvent) {
+  if (!keyboardShortcuts.value) {
+    return
+  }
+
   const items = navigableItems.value
   const count = items.length
 

app/components/Package/Playgrounds.vue

@@ -69,7 +69,13 @@ function closeDropdown() {
   focusedIndex.value = -1
 }
 
+const keyboardShortcuts = useKeyboardShortcuts()
+
 function handleKeydown(event: KeyboardEvent) {
+  if (!keyboardShortcuts.value) {
+    return
+  }
+
   if (!isOpen.value) {
     if (event.key === 'ArrowDown' || event.key === 'Enter' || event.key === ' ') {
       event.preventDefault()

app/pages/search.vue

@@ -463,7 +463,12 @@ function focusSearchInput() {
   searchInput?.focus()
 }
 
+const keyboardShortcuts = useKeyboardShortcuts()
+
 function handleResultsKeydown(e: KeyboardEvent) {
+  if (!keyboardShortcuts.value) {
+    return
+  }
   // If the active element is an input, navigate to exact match or wait for results
   if (e.key === 'Enter' && document.activeElement?.tagName === 'INPUT') {
     // Get value directly from input (not from route query, which may be debounced)