From 69a82e5e5c501e070f7aacc918bc57bff5f8a521 Mon Sep 17 00:00:00 2001
From: Eric Liu <ericyl.us@gmail.com>
Date: Thu, 26 Nov 2020 15:10:59 -0800
Subject: [PATCH] feat(header-search): deefault selectedResultIndex should be 0

Reset selectedResultIndex, value after dispatching.
---
 src/UIShell/HeaderSearch.svelte | 15 +++++++++++----
 tests/HeaderSearch.svelte       |  2 ++
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/src/UIShell/HeaderSearch.svelte b/src/UIShell/HeaderSearch.svelte
index b6e24bcd..72398e20 100644
--- a/src/UIShell/HeaderSearch.svelte
+++ b/src/UIShell/HeaderSearch.svelte
@@ -1,6 +1,8 @@
 <script>
   /**
    * @typedef {{ href: string; text: string; description?: string; }} HeaderSearchResult
+   * @event {any} active
+   * @event {any} inactive
    * @event {any} clear
    * @event {{ value: string; selectedResultIndex: number; selectedResult: HeaderSearchResult }} select
    */
@@ -21,7 +23,7 @@
   export let results = [];
 
   /** Specify the selected result index */
-  export let selectedResultIndex = -1;
+  export let selectedResultIndex = 0;
 
   import { createEventDispatcher } from "svelte";
   import Close20 from "carbon-icons-svelte/lib/Close20/Close20.svelte";
@@ -31,13 +33,19 @@
 
   let refSearch = null;
 
-  function selectResult() {
+  function reset() {
     active = false;
     value = "";
+    selectedResultIndex = 0;
+  }
+
+  function selectResult() {
     dispatch("select", { value, selectedResultIndex, selectedResult });
+    reset();
   }
 
   $: if (active && ref) ref.focus();
+  $: dispatch(active ? "active" : "inactive");
   $: selectedResult = results[selectedResultIndex];
   $: selectedId = selectedResult
     ? `search-menuitem-${selectedResultIndex}`
@@ -243,8 +251,7 @@
       class:bx--header__action="{true}"
       class:hidden="{!active}"
       on:click="{() => {
-        active = false;
-        value = '';
+        reset();
         dispatch('clear');
       }}"
     >
diff --git a/tests/HeaderSearch.svelte b/tests/HeaderSearch.svelte
index 6269db14..8ad09125 100644
--- a/tests/HeaderSearch.svelte
+++ b/tests/HeaderSearch.svelte
@@ -65,6 +65,8 @@
       bind:value
       bind:selectedResultIndex
       results="{results}"
+      on:active
+      on:inactive
       on:clear="{() => {
         console.log('on:clear');
       }}"