From ae53c3e6232e30d37fb33fbe4162c07a472b1a67 Mon Sep 17 00:00:00 2001
From: Nate Butler <iamnbutler@gmail.com>
Date: Fri, 16 Jun 2023 01:41:10 -0400
Subject: [PATCH] WIP: Start updating style trees to new `toggle()` format.

---
 styles/src/styleTree/commandPalette.ts | 62 +++++++++++++-------------
 styles/src/styleTree/contactList.ts    | 18 ++++----
 2 files changed, 42 insertions(+), 38 deletions(-)

diff --git a/styles/src/styleTree/commandPalette.ts b/styles/src/styleTree/commandPalette.ts
index 59e12945db864c1d486bfb68fb6dc349b6165ce6..cb6bb0503afcba951dc36c7443e407a199bfa873 100644
--- a/styles/src/styleTree/commandPalette.ts
+++ b/styles/src/styleTree/commandPalette.ts
@@ -1,40 +1,42 @@
 import { ColorScheme } from "../theme/colorScheme"
 import { withOpacity } from "../theme/color"
 import { text, background } from "./components"
-import { interactive, toggleable } from "../element"
+import { toggleable } from "../element"
 
 export default function commandPalette(colorScheme: ColorScheme) {
     let layer = colorScheme.highest
+
+    const key = toggleable({
+        base: {
+            text: text(layer, "mono", "variant", "default", { size: "xs" }),
+            cornerRadius: 2,
+            background: background(layer, "on"),
+            padding: {
+                top: 1,
+                bottom: 1,
+                left: 6,
+                right: 6,
+            },
+            margin: {
+                top: 1,
+                bottom: 1,
+                left: 2,
+            },
+        },
+        state: {
+            active: {
+                text: text(layer, "mono", "on", "default", { size: "xs" }),
+                background: withOpacity(background(layer, "on"), 0.2),
+            }
+        }
+    })
+
     return {
         keystrokeSpacing: 8,
-        key: toggleable(
-            interactive({
-                base: {
-                    text: text(layer, "mono", "variant", "default", {
-                        size: "xs",
-                    }),
-                    cornerRadius: 2,
-                    background: background(layer, "on"),
-                    padding: {
-                        top: 1,
-                        bottom: 1,
-                        left: 6,
-                        right: 6,
-                    },
-                    margin: {
-                        top: 1,
-                        bottom: 1,
-                        left: 2,
-                    },
-                },
-                state: { hovered: { cornerRadius: 4, padding: { top: 17 } } },
-            }),
-            {
-                default: {
-                    text: text(layer, "mono", "on", "default", { size: "xs" }),
-                    background: withOpacity(background(layer, "on"), 0.2),
-                },
-            }
-        ),
+        // TODO: This should be a Toggle<ContainedText> on the rust side so we don't have to do this
+        key: {
+            ...key.inactive,
+            active: key.active,
+        }
     }
 }
diff --git a/styles/src/styleTree/contactList.ts b/styles/src/styleTree/contactList.ts
index 2ac17a4584c222c477709b03fb4a512e8df2093a..1f38b9b37686856da29ec71ad8c3bc37b333aa51 100644
--- a/styles/src/styleTree/contactList.ts
+++ b/styles/src/styleTree/contactList.ts
@@ -71,8 +71,8 @@ export default function contactsPanel(colorScheme: ColorScheme) {
         },
         rowHeight: 28,
         sectionIconSize: 8,
-        headerRow: toggleable(
-            interactive({
+        headerRow: toggleable({
+            base: interactive({
                 base: {
                     ...text(layer, "mono", { size: "sm" }),
                     margin: { top: 14 },
@@ -86,13 +86,15 @@ export default function contactsPanel(colorScheme: ColorScheme) {
                     hovered: { background: background(layer, "default") },
                 }, // hack, we want headerRow to be interactive for whatever reason. It probably shouldn't be interactive in the first place.
             }),
-            {
-                default: {
-                    ...text(layer, "mono", "active", { size: "sm" }),
-                    background: background(layer, "active"),
-                },
+            state: {
+                active: {
+                    default: {
+                        ...text(layer, "mono", "active", { size: "sm" }),
+                        background: background(layer, "active"),
+                    },
+                }
             }
-        ),
+        }),
         leaveCall: interactive({
             base: {
                 background: background(layer),