From 9a2b7908979b88e4792ed7e55880d1e8a1325b72 Mon Sep 17 00:00:00 2001
From: Michael Freno <michaelt.freno@gmail.com>
Date: Wed, 11 Feb 2026 14:10:35 -0500
Subject: [PATCH] for consistency

---
 src/components/Selectable.tsx    | 41 ++++++++++++++++++++++++++++++++
 src/components/TabNavigation.tsx | 21 +++++++---------
 2 files changed, 49 insertions(+), 13 deletions(-)
 create mode 100644 src/components/Selectable.tsx

diff --git a/src/components/Selectable.tsx b/src/components/Selectable.tsx
new file mode 100644
index 0000000..77436e7
--- /dev/null
+++ b/src/components/Selectable.tsx
@@ -0,0 +1,41 @@
+import { useTheme } from "@/context/ThemeContext";
+import type { JSXElement } from "solid-js";
+import type { BoxOptions, TextOptions } from "@opentui/core";
+
+export function SelectableBox({
+  selected,
+  children,
+  ...props
+}: {
+  selected: () => boolean;
+  children: JSXElement;
+} & BoxOptions) {
+  const { theme } = useTheme();
+
+  return (
+    <box
+      borderColor={selected() ? theme.surface : theme.border}
+      backgroundColor={selected() ? theme.primary : theme.surface}
+      {...props}
+    >
+      {children}
+    </box>
+  );
+}
+
+export function SelectableText({
+  selected,
+  children,
+  ...props
+}: {
+  selected: () => boolean;
+  children: JSXElement;
+} & TextOptions) {
+  const { theme } = useTheme();
+
+  return (
+    <text fg={selected() ? theme.surface : theme.text} {...props}>
+      {children}
+    </text>
+  );
+}
diff --git a/src/components/TabNavigation.tsx b/src/components/TabNavigation.tsx
index 91e7358..d8573eb 100644
--- a/src/components/TabNavigation.tsx
+++ b/src/components/TabNavigation.tsx
@@ -1,6 +1,7 @@
 import { useTheme } from "@/context/ThemeContext";
 import { TABS } from "@/utils/navigation";
 import { For } from "solid-js";
+import { SelectableBox, SelectableText } from "@/components/Selectable";
 
 interface TabNavigationProps {
   activeTab: TABS;
@@ -29,24 +30,18 @@ export function TabNavigation(props: TabNavigationProps) {
     >
       <For each={tabs}>
         {(tab) => (
-          <box
+          <SelectableBox
             border
-            borderColor={theme.border}
+            selected={() => tab.id == props.activeTab}
             onMouseDown={() => props.onTabSelect(tab.id)}
-            style={{
-              backgroundColor:
-                tab.id == props.activeTab ? theme.primary : "transparent",
-            }}
           >
-            <text
-              style={{
-                fg: tab.id == props.activeTab ? "white" : theme.text,
-                alignSelf: "center",
-              }}
+            <SelectableText
+              selected={() => tab.id == props.activeTab}
+              alignSelf="center"
             >
               {tab.label}
-            </text>
-          </box>
+            </SelectableText>
+          </SelectableBox>
         )}
       </For>
     </box>