From 31092df5df7685217b09065ade07a2fe84d32f36 Mon Sep 17 00:00:00 2001
From: Dane Everitt <dane@daneeveritt.com>
Date: Tue, 11 Sep 2018 21:32:01 -0700
Subject: [PATCH] Show loading indicator when searching

---
 .../scripts/components/core/Navigation.vue    | 49 +++++++++++++------
 .../assets/styles/components/navigation.css   |  2 +-
 .../assets/styles/components/spinners.css     |  4 ++
 3 files changed, 39 insertions(+), 16 deletions(-)

diff --git a/resources/assets/scripts/components/core/Navigation.vue b/resources/assets/scripts/components/core/Navigation.vue
index c508819ee..5f43d2555 100644
--- a/resources/assets/scripts/components/core/Navigation.vue
+++ b/resources/assets/scripts/components/core/Navigation.vue
@@ -7,28 +7,42 @@
         </div>
         <div class="search-box flex-none" v-if="$route.name !== 'dashboard'">
             <input type="text" class="search-input" id="searchInput" placeholder="Search..."
-                   :class="{ 'has-search-results': servers.length > 0 && searchActive }"
+                   :class="{ 'has-search-results': (servers.length > 0 || loadingResults) && searchActive }"
                    v-on:focus="searchActive = true"
                    v-on:blur="searchActive = false"
                    v-on:input="search"
                    v-model="searchTerm"
             />
-            <div class="search-results select-none" :class="{ 'hidden': servers.length === 0 || !searchActive }">
-                <router-link
-                        v-for="server in servers"
-                        :key="server.identifier"
-                        :to="{ name: 'server', params: { id: server.identifier } }"
-                >
-                    <div class="flex items-center">
-                        <div class="flex-1">
-                            <span class="font-bold text-grey-darkest">{{ server.name }}</span><br />
-                            <span class="font-light text-grey-dark text-sm" v-if="server.description.length > 0">{{ server.description }}</span>
+            <div class="search-results select-none" :class="{ 'hidden': (servers.length === 0 && !loadingResults) || !searchActive }">
+                <div v-if="loadingResults">
+                    <a href="#">
+                        <div class="flex items-center">
+                            <div class="flex-1">
+                                <span class="text-sm text-grey-darker">Loading...</span>
+                            </div>
+                            <div class="flex-none">
+                                <span class="spinner spinner-relative"></span>
+                            </div>
                         </div>
-                        <div class="flex-none">
-                            <span class="pillbox bg-indigo">{{ server.node }}</span>
+                    </a>
+                </div>
+                <div v-else>
+                    <router-link
+                            v-for="server in servers"
+                            :key="server.identifier"
+                            :to="{ name: 'server', params: { id: server.identifier } }"
+                    >
+                        <div class="flex items-center">
+                            <div class="flex-1">
+                                <span class="font-bold text-grey-darkest">{{ server.name }}</span><br />
+                                <span class="font-light text-grey-dark text-sm" v-if="server.description.length > 0">{{ server.description }}</span>
+                            </div>
+                            <div class="flex-none">
+                                <span class="pillbox bg-indigo">{{ server.node }}</span>
+                            </div>
                         </div>
-                    </div>
-                </router-link>
+                    </router-link>
+                </div>
             </div>
         </div>
         <div class="menu flex-none">
@@ -69,6 +83,7 @@
 
         data: function () {
             return {
+                loadingResults: false,
                 searchActive: false,
             };
         },
@@ -88,6 +103,7 @@
         methods: {
             search: debounce(function () {
                 if (this.searchTerm.length > 3) {
+                    this.loadingResults = true;
                     this.gatherSearchResults(this.searchTerm);
                 }
             }, 500),
@@ -106,6 +122,9 @@
                                 this.error(error.detail);
                             });
                         }
+                    })
+                    .then(() => {
+                        this.loadingResults = false;
                     });
             },
 
diff --git a/resources/assets/styles/components/navigation.css b/resources/assets/styles/components/navigation.css
index cdbe5edf0..74a50d3a6 100644
--- a/resources/assets/styles/components/navigation.css
+++ b/resources/assets/styles/components/navigation.css
@@ -37,7 +37,7 @@
             @apply .absolute .bg-grey-lightest .border .border-blue-darkest .border-t-0 .rounded .rounded-t-none .p-2 .mx-3 .z-50;
             width: 26rem;
 
-            & > a {
+            & a {
                 @apply .block .no-underline .p-2 .rounded;
 
                 &:hover {
diff --git a/resources/assets/styles/components/spinners.css b/resources/assets/styles/components/spinners.css
index 4232ad130..1bdaa08fe 100644
--- a/resources/assets/styles/components/spinners.css
+++ b/resources/assets/styles/components/spinners.css
@@ -15,6 +15,10 @@
     left: calc(50% - (1em / 2));
   }
 
+  &.spinner-relative:after {
+    @apply .relative;
+  }
+
   &.spinner-xl:after {
     @apply .h-16 .w-16;
     left: calc(50% - (4rem / 2));