Skip to content
This repository was archived by the owner on Apr 11, 2026. It is now read-only.

Commit 380cca6

Browse files
z23ccclaude
andcommitted
style(design): /design-review fixes — FINDING-001..004
- Settings: format JSON config as readable key-value pairs instead of raw output - Memory: warm empty state with icon and guidance copy - Dashboard: epic card titles 14px→15px, weight 500→600 for hierarchy - TaskSidebar: close button 26x26px → 44x44px (WCAG touch target) Design audit: Design Score B, AI Slop Score A. Responsive (mobile/tablet/desktop) is genuinely well-designed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 4330880 commit 380cca6

4 files changed

Lines changed: 37 additions & 10 deletions

File tree

frontend/src/components/TaskSidebar.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,8 @@ export default function TaskSidebar() {
9191
</h2>
9292
<button
9393
onClick={close}
94-
className="p-1 rounded hover:bg-bg-tertiary text-text-muted hover:text-text-primary transition-colors"
94+
aria-label="Close task details"
95+
className="p-1 rounded hover:bg-bg-tertiary text-text-muted hover:text-text-primary transition-colors min-h-[44px] min-w-[44px] flex items-center justify-center"
9596
>
9697
<X size={18} />
9798
</button>

frontend/src/pages/Dashboard.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ export default function Dashboard() {
248248
className="block rounded-lg border border-border bg-bg-secondary p-4 hover:border-accent transition-colors"
249249
>
250250
<div className="flex items-start justify-between gap-2 mb-3">
251-
<h3 className="text-sm font-medium truncate">{epic.title}</h3>
251+
<h3 className="text-[15px] font-semibold truncate">{epic.title}</h3>
252252
<Badge status={epicStatusKey(epic.status)} label={epic.status.replace("_", " ")} />
253253
</div>
254254
<ProgressBar tasks={epic.tasks} />

frontend/src/pages/Memory.tsx

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,15 @@ export default function Memory() {
7777
)}
7878

7979
{!isLoading && entries.length === 0 && (
80-
<div className="flex-1 flex items-center justify-center">
81-
<p className="text-text-muted text-sm">No memory entries found.</p>
80+
<div className="flex-1 flex flex-col items-center justify-center gap-3 text-center">
81+
<div className="text-4xl opacity-40">🧠</div>
82+
<p className="text-text-primary text-sm font-medium">
83+
No memory entries yet
84+
</p>
85+
<p className="text-text-muted text-xs max-w-md">
86+
Memory accumulates automatically as agents work. Lessons from
87+
completed tasks, pitfalls, and conventions will appear here.
88+
</p>
8289
</div>
8390
)}
8491

frontend/src/pages/Settings.tsx

Lines changed: 25 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,32 @@ export default function Settings() {
3737
{entries.map(([key, value]) => (
3838
<tr
3939
key={key}
40-
className="border-t border-border bg-bg-secondary"
40+
className="border-t border-border bg-bg-secondary align-top"
4141
>
42-
<td className="px-4 py-2 font-mono text-accent">{key}</td>
43-
<td className="px-4 py-2 text-text-primary">
44-
{typeof value === "object"
45-
? JSON.stringify(value)
46-
: String(value)}
42+
<td className="px-4 py-3 font-mono text-accent whitespace-nowrap">
43+
{key}
44+
</td>
45+
<td className="px-4 py-3 text-text-primary">
46+
{typeof value === "object" && value !== null ? (
47+
<div className="flex flex-col gap-1 text-xs">
48+
{Object.entries(value as Record<string, unknown>).map(
49+
([k, v]) => (
50+
<div key={k} className="flex gap-2">
51+
<span className="text-text-muted font-mono">
52+
{k}:
53+
</span>
54+
<span className="font-mono">
55+
{typeof v === "object"
56+
? JSON.stringify(v)
57+
: String(v)}
58+
</span>
59+
</div>
60+
),
61+
)}
62+
</div>
63+
) : (
64+
<span className="font-mono text-xs">{String(value)}</span>
65+
)}
4766
</td>
4867
</tr>
4968
))}

0 commit comments

Comments
 (0)