From 0494ce8a8631896e88cf77a98ea185b4521cdf00 Mon Sep 17 00:00:00 2001
From: Sam Becker <sam@sambecker.com>
Date: Wed, 12 Feb 2025 23:09:13 -0600
Subject: [PATCH] Add insight expand interaction

---
 src/admin/AdminAppInsightsClient.tsx | 34 ++++++++++++------------
 src/components/ScoreCardRow.tsx      | 39 ++++++++++++++++++----------
 2 files changed, 42 insertions(+), 31 deletions(-)

diff --git a/src/admin/AdminAppInsightsClient.tsx b/src/admin/AdminAppInsightsClient.tsx
index 52fae8fa..2a440976 100644
--- a/src/admin/AdminAppInsightsClient.tsx
+++ b/src/admin/AdminAppInsightsClient.tsx
@@ -58,7 +58,6 @@ export default function AdminAppInsightsClient({
   return (
     <div className={clsx(
       'flex flex-col items-center w-full',
-      'mt-2 mb-1',
     )}>
       <div className={clsx(
         'w-full sm:w-[80%] lg:w-[60%]',
@@ -72,28 +71,27 @@ export default function AdminAppInsightsClient({
                 className="translate-x-[0.5px] text-amber-600"
               />
             }
-          >
-            This fork is 9 commits behind
-          </ScoreCardRow>
+            content="This fork is 9 commits behind"
+            additionalContent="This fork is 9 commits behind"
+          />
           <ScoreCardRow
-            icon={
-              <PiWarningBold
-                size={17}
-                className="translate-x-[0.5px] text-amber-600"
-              />}
-          >
-            Consider enabling rate limiting to mitigate AI abuse
-          </ScoreCardRow>
+            icon={<PiWarningBold
+              size={17}
+              className="translate-x-[0.5px] text-amber-600"
+            />}
+            content="Consider enabling rate limiting to mitigate AI abuse"
+            // eslint-disable-next-line max-len
+            additionalContent="Consider enabling rate limiting to mitigate AI abuse"
+          />
           <ScoreCardRow
             icon={<IconGrSync />}
-          >
-            Consider forking this repository to receive new features and fixes
-          </ScoreCardRow>
+            // eslint-disable-next-line max-len
+            content="Consider forking this repository to receive new features and fixes"
+          />
           <ScoreCardRow
             icon={<HiSparkles />}
-          >
-            Enable AI text generation in the app configuration
-          </ScoreCardRow>
+            content="Enable AI text generation in the app configuration"
+          />
         </ScoreCard>
         {renderTitle('Code Observability')}
         {(fork || debug) &&
diff --git a/src/components/ScoreCardRow.tsx b/src/components/ScoreCardRow.tsx
index 716d4e47..744ef1a8 100644
--- a/src/components/ScoreCardRow.tsx
+++ b/src/components/ScoreCardRow.tsx
@@ -1,29 +1,42 @@
 import { clsx } from 'clsx';
-import { ReactNode } from 'react';
+import { ReactNode, useState } from 'react';
+import { FaMinus, FaPlus } from 'react-icons/fa6';
 export default function ScoreCardRow({
   icon,
-  children,
-  details,
+  content,
+  additionalContent,
 }: {
   icon: ReactNode
-  children: ReactNode
-  details?: string
+  content: ReactNode
+  additionalContent?: ReactNode
 }) {
+  const [isExpanded, setIsExpanded] = useState(false);
   return (
     <div className={clsx(
-      'flex items-center gap-4',
+      'flex gap-4',
       'px-4 py-2',
     )}>
-      <div className="flex items-center gap-2 shrink-0">
+      <div className="pt-[3px] shrink-0">
         {icon}
       </div>
-      <div>
-        {children}
+      <div className="grow space-y-4">
+        <div className="text-main">
+          {content}
+        </div>
+        {isExpanded &&
+          <div className="text-sm text-medium">
+            {additionalContent}
+          </div>}
       </div>
-      {details &&
-        <span className="text-sm text-gray-500">
-          {details}
-        </span>}
+      {additionalContent && <button
+        type="button"
+        onClick={() => setIsExpanded(!isExpanded)}
+        className="px-[9px] self-start -mr-1"
+      >
+        {isExpanded
+          ? <FaMinus />
+          : <FaPlus />}
+      </button>}
     </div>
   );
 }