From 7e04b635cd39d4e35a8a917827daf7109b8c5465 Mon Sep 17 00:00:00 2001
From: Tobias Berger <tobi.berger13@gmail.com>
Date: Mon, 27 Sep 2021 14:36:40 +0200
Subject: [PATCH] Remember that stores exist

---
 src/components/Header.svelte                  |  4 +
 src/components/Log.svelte                     | 28 +++---
 .../ResourceTable/ResourceGroup.svelte        | 48 ++++++++++
 .../{ => ResourceTable}/ResourceTable.svelte  | 92 +++++++------------
 src/components/Wrapper.svelte                 |  2 +-
 src/shark/Log.ts                              | 29 ++++--
 6 files changed, 119 insertions(+), 84 deletions(-)
 create mode 100644 src/components/ResourceTable/ResourceGroup.svelte
 rename src/components/{ => ResourceTable}/ResourceTable.svelte (51%)

diff --git a/src/components/Header.svelte b/src/components/Header.svelte
index 1ee615f..44e0c82 100644
--- a/src/components/Header.svelte
+++ b/src/components/Header.svelte
@@ -81,6 +81,10 @@
 
     pointer-events: none;
 
+    > span {
+      font-weight: bold;
+    }
+
     > ul {
       text-align: center;
       font-weight: bold;
diff --git a/src/components/Log.svelte b/src/components/Log.svelte
index 9d35236..9d18853 100644
--- a/src/components/Log.svelte
+++ b/src/components/Log.svelte
@@ -1,45 +1,45 @@
+<svelte:options immutable />
+
 <script lang="ts">
+  import type { Writable } from "svelte/store";
+
   import { Message, MessageType } from "../shark/Message";
   import { SharkGame } from "../shark/SharkGame";
 
-  export let messages: Message[];
+  export let messages: Writable<Message[]>;
 
-  function addMessage(
-    message: string,
-    messageType: MessageType = MessageType.message
-  ) {
-    messages = [...messages, new Message(message, messageType)];
+  // Shorthand
+  function addMessage(message: string, messageType?: MessageType) {
+    SharkGame.Log.addMessage(message, messageType);
   }
 
   $: logLength = SharkGame.Settings.logLength.current;
-  $: displaymessages = messages.slice(-logLength).reverse();
+  $: displayMessages = $messages.slice(-logLength).reverse();
 </script>
 
 <div id="log">
   <button
     on:click={() => {
-      addMessage("message" + messages.length);
+      addMessage("message" + $messages.length);
     }}>Add Message</button
   >
   <button
     on:click={() => {
-      addMessage("warn" + messages.length, MessageType.warning);
+      addMessage("warn" + $messages.length, MessageType.warning);
     }}>Add Warning</button
   >
   <button
     on:click={() => {
-      addMessage("error" + messages.length, MessageType.error);
+      addMessage("error" + $messages.length, MessageType.error);
     }}>Add Error</button
   >
   <button
     on:click={() => {
-      Message.even = true;
-      messages = [];
-      addMessage("Log cleared");
+      SharkGame.Log.reset();
     }}>Clear log</button
   >
   <ol>
-    {#each displaymessages as message, i}
+    {#each displayMessages as message, i}
       {#if i < logLength}
         <li class={(message.isEven ? "even" : "odd") + " " + message.type}>
           {message}
diff --git a/src/components/ResourceTable/ResourceGroup.svelte b/src/components/ResourceTable/ResourceGroup.svelte
new file mode 100644
index 0000000..b9b469d
--- /dev/null
+++ b/src/components/ResourceTable/ResourceGroup.svelte
@@ -0,0 +1,48 @@
+<svelte:options immutable />
+
+<script lang="ts">
+  import type { ResourceEntry } from "./ResourceTable.svelte";
+  export let categoryName: string;
+  export let collapsed: boolean;
+  export let resources: ResourceEntry[];
+</script>
+
+<tr on:click class="subhead" tabindex="0" role="button">
+  <td>{collapsed ? "▲" : "▼"}</td>
+  <td colspan="3"><h4>{categoryName}</h4> </td>
+</tr>
+{#if !collapsed}
+  {#each resources as [resourceName, resource], index}
+    <tr class={index % 2 === 0 ? "even" : "odd"}>
+      <td colspan="2">{resourceName}</td>
+      <td>{resource.amount}</td>
+      <td>{Math.round(100 * resource.change) / 100}/s</td>
+    </tr>
+  {/each}
+{/if}
+
+<style lang="scss">
+  tr {
+    &.even {
+      background-color: var(--color-med);
+    }
+    &.odd {
+      background-color: var(--color-light);
+    }
+    &.subhead {
+      cursor: pointer;
+      user-select: none;
+
+      background-color: var(--color-dark);
+
+      border: 1px solid var(--color-lighter);
+
+      > td > h4 {
+        margin: 0.3em;
+      }
+      &:hover > td > h4 {
+        text-shadow: 0 0 8px var(--color-lighter);
+      }
+    }
+  }
+</style>
diff --git a/src/components/ResourceTable.svelte b/src/components/ResourceTable/ResourceTable.svelte
similarity index 51%
rename from src/components/ResourceTable.svelte
rename to src/components/ResourceTable/ResourceTable.svelte
index 40472c5..e4bc3f3 100644
--- a/src/components/ResourceTable.svelte
+++ b/src/components/ResourceTable/ResourceTable.svelte
@@ -1,4 +1,7 @@
-<script lang="ts">
+<svelte:options immutable={true} />
+
+<script context="module" lang="ts">
+  export type ResourceEntry = [string, typeof TEST_RESOURCES["Shark"]];
   const TEST_RESOURCES = {
     Shark: {
       amount: 100,
@@ -51,31 +54,38 @@
       change: (Math.random() - 0.5) * 30,
     },
   };
+</script>
 
+<script lang="ts">
+  import ResourceGroup from "./ResourceGroup.svelte";
   let collapsed: string[] = [];
-  $: grouped = Object.entries(resourceGroups());
+  $: groups = resourceGroups();
 
-  type Entry = [string, typeof TEST_RESOURCES["Shark"]];
   function resourceGroups() {
-    return Object.entries(TEST_RESOURCES).reduce(
-      (reduced: { [key: string]: Entry[] }, [resourceName, resource]) => {
-        if (reduced[resource.category] === undefined) {
-          reduced[resource.category] = [];
-        }
-        reduced[resource.category] = [
-          ...reduced[resource.category],
-          [resourceName, resource],
-        ];
+    return Object.entries(
+      Object.entries(TEST_RESOURCES).reduce(
+        (
+          reduced: { [key: string]: ResourceEntry[] },
+          [resourceName, resource]
+        ) => {
+          if (reduced[resource.category] === undefined) {
+            reduced[resource.category] = [];
+          }
+          reduced[resource.category] = [
+            ...reduced[resource.category],
+            [resourceName, resource],
+          ];
 
-        return reduced;
-      },
-      {}
+          return reduced;
+        },
+        {}
+      )
     );
   }
 
   function toggleCollapsed(categoryName: string) {
     if (collapsed.includes(categoryName)) {
-      collapsed = collapsed.filter((group) => group !== categoryName);
+      collapsed = collapsed.filter((category) => category !== categoryName);
     } else {
       collapsed = [...collapsed, categoryName];
     }
@@ -91,25 +101,13 @@
       <col style="width: 20%;" />
     </colgroup>
     <tbody>
-      {#each grouped as [categoryName, resources]}
-        <tr
+      {#each groups as [categoryName, resources]}
+        <ResourceGroup
           on:click={() => toggleCollapsed(categoryName)}
-          class="subhead"
-          tabindex="0"
-          role="button"
-        >
-          <td>{collapsed.includes(categoryName) ? "▲" : "▼"}</td>
-          <td colspan="3"><h4>{categoryName}</h4> </td>
-        </tr>
-        {#if !collapsed.includes(categoryName)}
-          {#each resources as [resourceName, resource], index}
-            <tr class={index % 2 === 0 ? "even" : "odd"}>
-              <td colspan="2">{resourceName}</td>
-              <td>{resource.amount}</td>
-              <td>{Math.round(100 * resource.change) / 100}/s</td>
-            </tr>
-          {/each}
-        {/if}
+          {categoryName}
+          {resources}
+          collapsed={collapsed.includes(categoryName)}
+        />
       {/each}
     </tbody>
   </table>
@@ -126,32 +124,6 @@
     > table {
       width: 100%;
       border-collapse: collapse;
-
-      > tbody {
-        > tr {
-          &.even {
-            background-color: var(--color-med);
-          }
-          &.odd {
-            background-color: var(--color-light);
-          }
-          &.subhead {
-            cursor: pointer;
-            user-select: none;
-
-            background-color: var(--color-dark);
-
-            border: 1px solid var(--color-lighter);
-
-            > td > h4 {
-              margin: 0.3em;
-            }
-            &:hover > td > h4 {
-              text-shadow: 0 0 8px var(--color-lighter);
-            }
-          }
-        }
-      }
     }
   }
 </style>
diff --git a/src/components/Wrapper.svelte b/src/components/Wrapper.svelte
index 811048f..06df4b5 100644
--- a/src/components/Wrapper.svelte
+++ b/src/components/Wrapper.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
   import { SharkGame } from "../shark/SharkGame";
   import Log from "./Log.svelte";
-  import ResourceTable from "./ResourceTable.svelte";
+  import ResourceTable from "./ResourceTable/ResourceTable.svelte";
 </script>
 
 <main>
diff --git a/src/shark/Log.ts b/src/shark/Log.ts
index 6c7892c..9c694b3 100644
--- a/src/shark/Log.ts
+++ b/src/shark/Log.ts
@@ -1,15 +1,26 @@
-import type { Message } from "./Message";
-import { SharkGame } from "./SharkGame";
+import { writable } from "svelte/store";
+
+import { Message, MessageType } from "./Message";
+import { Settings } from "./Settings";
 import { StaticClass } from "./StaticClass";
 
 export class Log extends StaticClass {
-  static #messages: Message[] = [];
-  static get messages(): Message[] {
-    return this.#messages;
-  }
-  static set messages(messages: Message[]) {
-    this.#messages = messages.slice(
-      -Math.max(...SharkGame.Settings.logLength.options)
+  static messages = writable<Message[]>([]);
+
+  static addMessage(
+    message: string,
+    messageType: MessageType = MessageType.message
+  ): void {
+    this.messages.update((oldMessages) =>
+      [...oldMessages, new Message(message, messageType)].slice(
+        -Math.max(...Settings.logLength.options)
+      )
     );
   }
+
+  static reset(): void {
+    Message.even = true;
+    this.messages.set([]);
+    this.addMessage("Log cleared");
+  }
 }