Skip to content

Commit 3ab55fc

Browse files
Refactor SessionCard layout to vertically stack checkbox with busy indicator
1 parent 2850de0 commit 3ab55fc

1 file changed

Lines changed: 19 additions & 17 deletions

File tree

frontend/src/components/session/SessionCard.tsx

Lines changed: 19 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -38,27 +38,29 @@ export const SessionCard = ({
3838
>
3939
<div className="flex items-start justify-between gap-2">
4040
<div className="flex items-start gap-2 flex-1 min-w-0">
41-
<Checkbox
42-
checked={isSelected}
43-
onCheckedChange={(checked) => {
44-
onToggleSelection(checked === true);
45-
}}
46-
onClick={(e) => {
47-
e.stopPropagation();
48-
}}
49-
className="w-5 h-5 flex-shrink-0 mt-0.5"
50-
/>
41+
<div className="flex flex-col items-center gap-3 flex-shrink-0">
42+
<Checkbox
43+
checked={isSelected}
44+
onCheckedChange={(checked) => {
45+
onToggleSelection(checked === true);
46+
}}
47+
onClick={(e) => {
48+
e.stopPropagation();
49+
}}
50+
className="w-5 h-5 flex-shrink-0"
51+
/>
52+
{isSessionBusy && (
53+
<span className="relative flex h-2.5 w-2.5 flex-shrink-0">
54+
<span className="animate-ping absolute inline-flex h-full w-full rounded-full bg-green-400 opacity-75"></span>
55+
<span className="relative inline-flex rounded-full h-2.5 w-2.5 bg-green-500"></span>
56+
</span>
57+
)}
58+
</div>
5159
<div className="flex-1 min-w-0">
5260
<div className="flex items-center gap-2">
5361
<h3 className="text-base font-semibold text-orange-600 dark:text-orange-400 truncate">
5462
{session.title || "Untitled Session"}
5563
</h3>
56-
{isSessionBusy && (
57-
<span className="relative flex h-2.5 w-2.5 flex-shrink-0">
58-
<span className="animate-ping absolute inline-flex h-full w-full rounded-full bg-green-400 opacity-75"></span>
59-
<span className="relative inline-flex rounded-full h-2.5 w-2.5 bg-green-500"></span>
60-
</span>
61-
)}
6264
</div>
6365
<div className="flex items-center gap-3 mt-1 text-xs text-muted-foreground">
6466
<span className="flex items-center gap-1">
@@ -79,4 +81,4 @@ export const SessionCard = ({
7981
</div>
8082
</Card>
8183
);
82-
};
84+
};

0 commit comments

Comments
 (0)