From f12bfc2b8c023b495260347c8058fc87e6e307e5 Mon Sep 17 00:00:00 2001
From: Niek van der Maas <mail@niekvandermaas.nl>
Date: Thu, 23 Mar 2023 16:13:55 +0100
Subject: [PATCH] Cleaner code in main app

---
 src/App.svelte         |  7 ++-----
 src/lib/Sidebar.svelte | 18 ++++++++----------
 2 files changed, 10 insertions(+), 15 deletions(-)

diff --git a/src/App.svelte b/src/App.svelte
index 58501e3..ed314c9 100644
--- a/src/App.svelte
+++ b/src/App.svelte
@@ -6,10 +6,7 @@
   import Sidebar from './lib/Sidebar.svelte'
   import Footer from './lib/Footer.svelte'
 
-  import { apiKeyStorage, chatsStorage } from './lib/Storage.svelte'
-
-  $: sortedChats = $chatsStorage.sort((a, b) => b.id - a.id)
-  $: apiKey = $apiKeyStorage
+  import { apiKeyStorage } from './lib/Storage.svelte'
 
   // Check if the API key is passed in as a "key" query parameter - if so, save it
   const urlParams: URLSearchParams = new URLSearchParams(window.location.search)
@@ -24,7 +21,7 @@
   <div class="container is-fullhd">
     <div class="columns">
       <div class="column is-one-fifth">
-        <Sidebar bind:apiKey bind:sortedChats />
+        <Sidebar />
       </div>
       <div class="column is-four-fifths" id="content">
         {#key $location}
diff --git a/src/lib/Sidebar.svelte b/src/lib/Sidebar.svelte
index 572a24e..4338789 100644
--- a/src/lib/Sidebar.svelte
+++ b/src/lib/Sidebar.svelte
@@ -1,12 +1,10 @@
 <script lang="ts">
   import { params, replace } from 'svelte-spa-router'
 
-  import { clearChats } from './Storage.svelte'
+  import { apiKeyStorage, chatsStorage, clearChats } from './Storage.svelte'
   import { exportAsMarkdown } from './Export.svelte'
-  import type { Chat } from './Types.svelte'
 
-  export let sortedChats: Chat[]
-  export let apiKey: string
+  $: sortedChats = $chatsStorage.sort((a, b) => b.id - a.id)
 
   $: activeChatId = $params && $params.chatId ? parseInt($params.chatId) : undefined
 </script>
@@ -15,13 +13,13 @@
   <p class="menu-label">Chats</p>
   <ul class="menu-list">
     {#if sortedChats.length === 0}
-      <li><a href={'#'}>No chats yet...</a></li>
+      <li><a href={'#'} class="is-disabled">No chats yet...</a></li>
     {:else}
       <li>
         <ul>
           {#each sortedChats as chat}
             <li>
-              <a href={`#/chat/${chat.id}`} class:is-disabled={!apiKey} class:is-active={activeChatId === chat.id}
+              <a href={`#/chat/${chat.id}`} class:is-disabled={!$apiKeyStorage} class:is-active={activeChatId === chat.id}
                 >{chat.name || `Chat ${chat.id}`}</a
               >
             </li>
@@ -33,12 +31,12 @@
   <p class="menu-label">Actions</p>
   <ul class="menu-list">
     <li>
-      <a href={'#/'} class="panel-block" class:is-disabled={!apiKey} class:is-active={!activeChatId}
+      <a href={'#/'} class="panel-block" class:is-disabled={!$apiKeyStorage} class:is-active={!activeChatId}
         ><span class="greyscale mr-2">🔑</span> API key</a
       >
     </li>
     <li>
-      <a href={'#/chat/new'} class="panel-block" class:is-disabled={!apiKey}
+      <a href={'#/chat/new'} class="panel-block" class:is-disabled={!$apiKeyStorage}
         ><span class="greyscale mr-2">➕</span> New chat</a
       >
     </li>
@@ -46,7 +44,7 @@
       <a
         href={'#/'}
         class="panel-block"
-        class:is-disabled={!apiKey}
+        class:is-disabled={!$apiKeyStorage}
         on:click={() => {
           replace('#/').then(() => {
             clearChats()
@@ -59,7 +57,7 @@
         <a
           href={'#/'}
           class="panel-block"
-          class:is-disabled={!apiKey}
+          class:is-disabled={!apiKeyStorage}
           on:click|preventDefault={() => {
             if (activeChatId) {
               exportAsMarkdown(activeChatId)