From 417cc694836b3a158edc3004707c6504b7c60133 Mon Sep 17 00:00:00 2001
From: Simon Cambier <simon.cambier@protonmail.com>
Date: Fri, 26 Aug 2022 08:54:40 +0200
Subject: [PATCH] alt+up/down to navigate search history

---
 src/components/InputSearch.svelte |  5 ++++-
 src/components/ModalVault.svelte  | 34 +++++++++++++++++++++++++------
 src/modals.ts                     | 10 +++++++++
 3 files changed, 42 insertions(+), 7 deletions(-)

diff --git a/src/components/InputSearch.svelte b/src/components/InputSearch.svelte
index 7d70654..49706b9 100644
--- a/src/components/InputSearch.svelte
+++ b/src/components/InputSearch.svelte
@@ -11,7 +11,10 @@
   onMount(async () => {
     await tick()
     elInput.focus()
-    elInput.select()
+    setTimeout(() => {
+      // tick() is not working here?
+      elInput.select()
+    }, 0)
   })
 
   const debouncedOnInput = debounce(() => {
diff --git a/src/components/ModalVault.svelte b/src/components/ModalVault.svelte
index c314453..65e4777 100644
--- a/src/components/ModalVault.svelte
+++ b/src/components/ModalVault.svelte
@@ -1,5 +1,5 @@
 <script lang="ts" context="module">
-  let lastSearch = ''
+  let lastSearches: string[] = []
 </script>
 
 <script lang="ts">
@@ -17,10 +17,12 @@
 
   export let modal: OmnisearchVaultModal
   let selectedIndex = 0
+  let lastSearchIndex = 0
   let searchQuery: string
   let resultNotes: ResultNote[] = []
   let query: Query
   $: selectedNote = resultNotes[selectedIndex]
+  // $: lastSearch = lastSearches[lastSearchIndex]
 
   $: if (searchQuery) {
     updateResults()
@@ -30,7 +32,8 @@
 
   onMount(async () => {
     await reindexNotes()
-    searchQuery = lastSearch
+    searchQuery = lastSearches[lastSearchIndex]
+    console.log(lastSearches)
     eventBus.enable('vault')
     eventBus.on('vault', 'enter', openNoteAndCloseModal)
     eventBus.on('vault', 'shift-enter', createNoteAndCloseModal)
@@ -39,18 +42,32 @@
     eventBus.on('vault', 'tab', switchToInFileModal)
     eventBus.on('vault', 'arrow-up', () => moveIndex(-1))
     eventBus.on('vault', 'arrow-down', () => moveIndex(1))
+    eventBus.on('vault', 'prev-search-history', () => prevSearch())
+    eventBus.on('vault', 'next-search-history', () => nextSearch())
   })
 
   onDestroy(() => {
     eventBus.disable('vault')
   })
 
+  function prevSearch() {
+    if (++lastSearchIndex >= lastSearches.length) {
+      lastSearchIndex = 0
+    }
+    searchQuery = lastSearches[lastSearchIndex]
+  }
+  function nextSearch() {
+    if (--lastSearchIndex < 0) {
+      lastSearchIndex = lastSearches.length ? lastSearches.length - 1 : 0
+    }
+    searchQuery = lastSearches[lastSearchIndex]
+  }
+
   async function updateResults() {
     query = new Query(searchQuery)
     resultNotes = (await getSuggestions(query)).sort(
       (a, b) => b.score - a.score
     )
-    lastSearch = searchQuery
     selectedIndex = 0
     scrollIntoView()
   }
@@ -63,16 +80,21 @@
 
   function openNoteAndCloseModal(): void {
     if (!selectedNote) return
-    openNote(selectedNote)
+    openSearchResult(selectedNote)
     modal.close()
   }
 
   function openNoteInNewPane(): void {
     if (!selectedNote) return
-    openNote(selectedNote, true)
+    openSearchResult(selectedNote, true)
     modal.close()
   }
 
+  function openSearchResult(note: ResultNote, newPane = false) {
+    lastSearches.unshift(searchQuery)
+    openNote(note, newPane)
+  }
+
   async function createNoteAndCloseModal(): Promise<void> {
     try {
       await createNote(searchQuery)
@@ -146,7 +168,7 @@
 </script>
 
 <div class="modal-title">Omnisearch - Vault</div>
-<InputSearch value={lastSearch} on:input={e => (searchQuery = e.detail)} />
+<InputSearch value={searchQuery} on:input={e => (searchQuery = e.detail)} />
 
 <ModalContainer>
   {#each resultNotes as result, i}
diff --git a/src/modals.ts b/src/modals.ts
index 6cdf1ac..23ee139 100644
--- a/src/modals.ts
+++ b/src/modals.ts
@@ -88,6 +88,16 @@ abstract class OmnisearchModal extends Modal {
       e.preventDefault()
       eventBus.emit('tab') // Switch context
     })
+
+    // Search history
+    this.scope.register(['Alt'], 'ArrowDown', e => {
+      e.preventDefault()
+      eventBus.emit('next-search-history')
+    })
+    this.scope.register(['Alt'], 'ArrowUp', e => {
+      e.preventDefault()
+      eventBus.emit('prev-search-history')
+    })
   }
 }