From b0ab4e8d1235abbb4633ddd055c8de00f9808fb1 Mon Sep 17 00:00:00 2001
From: Kamran Ahmed <kamranahmed.se@gmail.com>
Date: Mon, 26 May 2025 18:54:03 +0100
Subject: [PATCH] Update progress marking UI

---
 .../RoadmapAIChat/RoadmapTopicList.tsx        |  2 +-
 .../RoadmapAIChat/UserProgressActionList.tsx  | 23 +++++++++++++++----
 src/styles/global.css                         |  6 +++++
 3 files changed, 26 insertions(+), 5 deletions(-)

diff --git a/src/components/RoadmapAIChat/RoadmapTopicList.tsx b/src/components/RoadmapAIChat/RoadmapTopicList.tsx
index bec395fd9..8029673a6 100644
--- a/src/components/RoadmapAIChat/RoadmapTopicList.tsx
+++ b/src/components/RoadmapAIChat/RoadmapTopicList.tsx
@@ -69,7 +69,7 @@ export function RoadmapTopicList(props: RoadmapTopicListProps) {
       {progressItemWithText.map((item) => (
         <button
           key={item.topicId}
-          className="rounded-lg border border-gray-200 bg-white p-1 px-1.5 text-left text-sm"
+          className="collapse-if-empty rounded-lg border border-gray-200 bg-white p-1 px-1.5 text-left text-sm"
           onClick={() => {
             onTopicClick?.(item.topicId, item.text);
           }}
diff --git a/src/components/RoadmapAIChat/UserProgressActionList.tsx b/src/components/RoadmapAIChat/UserProgressActionList.tsx
index 03991eab3..70047bd91 100644
--- a/src/components/RoadmapAIChat/UserProgressActionList.tsx
+++ b/src/components/RoadmapAIChat/UserProgressActionList.tsx
@@ -11,6 +11,7 @@ import { useToast } from '../../hooks/use-toast';
 import { Check, ChevronRightIcon, Loader2Icon } from 'lucide-react';
 import { CheckIcon } from '../ReactIcons/CheckIcon';
 import { httpPost } from '../../lib/query-http';
+import { cn } from '../../lib/classname';
 
 type UpdateUserProgress = {
   id: string;
@@ -268,7 +269,7 @@ function ProgressItem(props: ProgressItemProps) {
   const lastIndex = textParts.length - 1;
 
   return (
-    <div className="flex items-center justify-between gap-2 rounded-lg border border-gray-200 bg-white py-1 pr-1 pl-3">
+    <div className="flex min-h-[40px] items-center justify-between gap-2 rounded-lg border border-gray-200 bg-white py-1 pr-1 pl-3">
       <span className="flex items-center gap-1 truncate text-sm text-gray-500">
         {textParts.map((part, index) => {
           return (
@@ -287,7 +288,19 @@ function ProgressItem(props: ProgressItemProps) {
         <>
           {!isStreaming && (
             <button
-              className="flex shrink-0 items-center gap-1.5 rounded-md border border-gray-200 bg-gray-100 px-2 py-1 text-sm hover:border-black hover:bg-black hover:text-white disabled:pointer-events-none disabled:opacity-40"
+              className={cn(
+                `flex shrink-0 items-center gap-1.5 rounded-md border border-gray-200 px-2 py-1 text-sm disabled:pointer-events-none disabled:opacity-40`,
+                {
+                  'bg-green-100 hover:border-green-300 hover:bg-green-200':
+                    action === 'done',
+                  'bg-yellow-100 hover:border-yellow-300 hover:bg-yellow-200':
+                    action === 'learning',
+                  'bg-gray-800 text-white hover:border-black hover:bg-black':
+                    action === 'skipped',
+                  'bg-gray-100 hover:border-gray-300 hover:bg-gray-200':
+                    action === 'pending',
+                },
+              )}
               onClick={() => updateTopicStatus(action)}
               disabled={isStreaming || isUpdating || isBulkUpdating}
             >
@@ -295,9 +308,11 @@ function ProgressItem(props: ProgressItemProps) {
                 <Loader2Icon className="size-4 animate-spin" />
               )}
               {!isUpdating && !isBulkUpdating && (
-                <Check strokeWidth={3} className="size-4" />
+                <>
+                  <Check strokeWidth={3} className="size-4" />
+                  Mark it as {action}
+                </>
               )}
-              Mark it as {action}
             </button>
           )}
           {isStreaming && (
diff --git a/src/styles/global.css b/src/styles/global.css
index 73079c6e6..14fee8605 100644
--- a/src/styles/global.css
+++ b/src/styles/global.css
@@ -58,6 +58,12 @@
   scrollbar-width: none; /* Firefox */
 }
 
+@utility collapse-if-empty {
+  &:empty {
+    display: none;
+  }
+}
+
 svg {
   user-select: none;
 }