From d857cce5dc248a0238b1213369595ca761c47918 Mon Sep 17 00:00:00 2001
From: Adam French <adam.a.french@outlook.com>
Date: Wed, 25 Mar 2026 02:43:24 +0000
Subject: [PATCH] Consolidate OptionalLinkTable and ToggleLinkTable into
 LinkTable

LinkTable now supports variant (list/table) and optional title toggle, replacing the need for separate components. Updates all consumers to use the unified API.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 nginx/vue/src/components/util/LinkTable.vue   | 69 ++++++++++++++++---
 .../src/components/util/OptionalLinkTable.vue | 27 --------
 .../src/components/util/ToggleLinkTable.vue   | 28 --------
 nginx/vue/src/views/Bookmarks.vue             |  8 +--
 nginx/vue/src/views/home/Consumption.vue      |  4 +-
 nginx/vue/src/views/home/Favorites.vue        |  7 +-
 nginx/vue/src/views/home/Gym.vue              |  4 +-
 nginx/vue/src/views/home/Links.vue            |  2 +-
 8 files changed, 73 insertions(+), 76 deletions(-)
 delete mode 100644 nginx/vue/src/components/util/OptionalLinkTable.vue
 delete mode 100644 nginx/vue/src/components/util/ToggleLinkTable.vue

diff --git a/nginx/vue/src/components/util/LinkTable.vue b/nginx/vue/src/components/util/LinkTable.vue
index 8773c93..2e8ee63 100644
--- a/nginx/vue/src/components/util/LinkTable.vue
+++ b/nginx/vue/src/components/util/LinkTable.vue
@@ -1,20 +1,73 @@
 <script setup>
-import { computed } from "vue";
+import { ref } from "vue";
+import Link from "@/components/text/Link.vue";
+import ToggleHeader from "@/components/text/ToggleHeader.vue";
 
 const props = defineProps({
-    linkArr: {
+    items: {
         type: Array,
         required: true,
     },
+    variant: {
+        type: String,
+        default: "list",
+    },
+    title: {
+        type: String,
+        default: "",
+    },
 });
 
-const keys = ["name", "link"];
+const show = ref(false);
 </script>
 
 <template>
-    <a v-for="(row, rowIndex) in linkArr" :key="rowIndex" :href="row.link">
-        <p class="bdr-2 bg-bg_tertiary">
-            {{ row.name }}
-        </p>
-    </a>
+    <div v-if="title" class="h-fit w-fit">
+        <ToggleHeader v-model="show" class="justify-between flex">
+            {{ title }}
+        </ToggleHeader>
+        <template v-if="show">
+            <Link
+                v-if="variant === 'list'"
+                v-for="(item, i) in items"
+                :key="i"
+                :href="item.link"
+            >
+                <p class="bdr-2 bg-bg_tertiary">{{ item.name }}</p>
+            </Link>
+            <table v-else>
+                <tbody>
+                    <tr v-for="item in items" :key="item.id">
+                        <th>{{ item.type }}</th>
+                        <td v-if="item.link">
+                            <Link :href="item.link">{{ item.name }}</Link>
+                        </td>
+                        <td v-else>{{ item.name }}</td>
+                    </tr>
+                </tbody>
+            </table>
+        </template>
+    </div>
+    <template v-else>
+        <template v-if="variant === 'list'">
+            <Link
+                v-for="(item, i) in items"
+                :key="i"
+                :href="item.link"
+            >
+                <p class="bdr-2 bg-bg_tertiary">{{ item.name }}</p>
+            </Link>
+        </template>
+        <table v-else>
+            <tbody>
+                <tr v-for="item in items" :key="item.id">
+                    <th>{{ item.type }}</th>
+                    <td v-if="item.link">
+                        <Link :href="item.link">{{ item.name }}</Link>
+                    </td>
+                    <td v-else>{{ item.name }}</td>
+                </tr>
+            </tbody>
+        </table>
+    </template>
 </template>
diff --git a/nginx/vue/src/components/util/OptionalLinkTable.vue b/nginx/vue/src/components/util/OptionalLinkTable.vue
deleted file mode 100644
index 0c25bf5..0000000
--- a/nginx/vue/src/components/util/OptionalLinkTable.vue
+++ /dev/null
@@ -1,27 +0,0 @@
-<script setup>
-// Array will have the form
-// [ {type: string, name: string, link?: string}]
-
-const props = defineProps({
-    data: {
-        type: Array,
-        required: true,
-    },
-});
-
-const keys = ["type", "name", "link"];
-</script>
-
-<template>
-    <table>
-        <tbody>
-            <tr v-for="item in data" :key="item.id">
-                <th>{{ item.type }}</th>
-                <td v-if="item.link">
-                    <a :href="item.link">{{ item.name }}</a>
-                </td>
-                <td v-else>{{ item.name }}</td>
-            </tr>
-        </tbody>
-    </table>
-</template>
diff --git a/nginx/vue/src/components/util/ToggleLinkTable.vue b/nginx/vue/src/components/util/ToggleLinkTable.vue
deleted file mode 100644
index c953fac..0000000
--- a/nginx/vue/src/components/util/ToggleLinkTable.vue
+++ /dev/null
@@ -1,28 +0,0 @@
-<script setup>
-import ToggleHeader from "@/components/text/ToggleHeader.vue";
-
-import { ref } from "vue";
-import LinkTable from "@/components/util/LinkTable.vue";
-
-const props = defineProps({
-    linkArr: {
-        type: Array,
-        required: true,
-    },
-    title: {
-        type: String,
-        required: true,
-    },
-});
-
-const show_links = ref(false);
-</script>
-
-<template>
-    <div class="h-fit w-fit">
-        <ToggleHeader v-model="show_links" class="justify-between flex"
-            >{{ title }}
-        </ToggleHeader>
-        <LinkTable v-if="show_links" :linkArr="props.linkArr" />
-    </div>
-</template>
diff --git a/nginx/vue/src/views/Bookmarks.vue b/nginx/vue/src/views/Bookmarks.vue
index 151c4c0..fe6fa5e 100644
--- a/nginx/vue/src/views/Bookmarks.vue
+++ b/nginx/vue/src/views/Bookmarks.vue
@@ -1,7 +1,5 @@
 <script setup>
-import { ref } from "vue";
-
-import ToggleLinkTable from "@/components/util/ToggleLinkTable.vue";
+import LinkTable from "@/components/util/LinkTable.vue";
 
 const links = [
     [
@@ -245,11 +243,11 @@ const links = [
             class="a4page-portrait bdr-1 flex flex-row flex-wrap overflow-x-auto gap-1"
         >
             <div class="w-full h-fit">
-                <ToggleLinkTable
+                <LinkTable
                     class="flex flex-col flex-wrap"
                     v-for="link in links"
                     :title="link[0]"
-                    :linkArr="link[1]"
+                    :items="link[1]"
                 />
             </div>
         </div>
diff --git a/nginx/vue/src/views/home/Consumption.vue b/nginx/vue/src/views/home/Consumption.vue
index 3af48fb..b4b1551 100644
--- a/nginx/vue/src/views/home/Consumption.vue
+++ b/nginx/vue/src/views/home/Consumption.vue
@@ -1,6 +1,6 @@
 <script setup>
 import AutoScroll from "@/components/util/AutoScroll.vue";
-import OptionalLinkTable from "@/components/util/OptionalLinkTable.vue";
+import LinkTable from "@/components/util/LinkTable.vue";
 import Header from "@/components/text/Header.vue";
 
 import { useActivityStore } from "@/stores/activity";
@@ -12,7 +12,7 @@ const activityStore = useActivityStore();
     <div class="flex flex-col items-center">
         <Header>Consumption</Header>
         <AutoScroll class="flex-1 w-full">
-            <OptionalLinkTable class="w-full" :data="activityStore.activity" />
+            <LinkTable variant="table" class="w-full" :items="activityStore.activity" />
         </AutoScroll>
     </div>
 </template>
diff --git a/nginx/vue/src/views/home/Favorites.vue b/nginx/vue/src/views/home/Favorites.vue
index 17b52a2..ecd68f2 100644
--- a/nginx/vue/src/views/home/Favorites.vue
+++ b/nginx/vue/src/views/home/Favorites.vue
@@ -1,6 +1,6 @@
 <script setup>
 import Header from "@/components/text/Header.vue";
-import OptionalLinkTable from "@/components/util/OptionalLinkTable.vue";
+import LinkTable from "@/components/util/LinkTable.vue";
 import AutoScroll from "@/components/util/AutoScroll.vue";
 
 import { useFavoritesStore } from "@/stores/favorites";
@@ -12,9 +12,10 @@ const favoritesStore = useFavoritesStore();
     <div class="flex flex-col items-center">
         <Header>favs</Header>
         <AutoScroll class="w-full flex-1">
-            <OptionalLinkTable
+            <LinkTable
+                variant="table"
                 class="w-full"
-                :data="favoritesStore.favorites"
+                :items="favoritesStore.favorites"
             />
         </AutoScroll>
     </div>
diff --git a/nginx/vue/src/views/home/Gym.vue b/nginx/vue/src/views/home/Gym.vue
index bcdaa55..001fabc 100644
--- a/nginx/vue/src/views/home/Gym.vue
+++ b/nginx/vue/src/views/home/Gym.vue
@@ -1,6 +1,6 @@
 <script setup>
 import Header from "@/components/text/Header.vue";
-import OptionalLinkTable from "@/components/util/OptionalLinkTable.vue";
+import LinkTable from "@/components/util/LinkTable.vue";
 const gym = [
     { name: "Row", type: "30 min" },
     { name: "Run", type: "5k" },
@@ -14,7 +14,7 @@ const gym = [
         <p>I'm not a gym geek</p>
         <p>4/7 days I do:</p>
         <div class="overflow-scroll w-full border-box">
-            <OptionalLinkTable class="w-full" :data="gym" />
+            <LinkTable variant="table" class="w-full" :items="gym" />
         </div>
     </div>
 </template>
diff --git a/nginx/vue/src/views/home/Links.vue b/nginx/vue/src/views/home/Links.vue
index 1d9697f..9724b21 100644
--- a/nginx/vue/src/views/home/Links.vue
+++ b/nginx/vue/src/views/home/Links.vue
@@ -24,7 +24,7 @@ const social_links = [
             <RouterTable :linkArr="site_links" />
         </div>
         <div class="flex flex-col gap-1">
-            <LinkTable :linkArr="social_links" />
+            <LinkTable :items="social_links" />
         </div>
     </div>
 </template>