fix: add sessions empty state guidance#9
Conversation
|
Thanks for the clean PR, @GodHad — really appreciate this being your first contribution to the repo. You hit the spec exactly: caught both empty-state render paths (table + virtualized), used the --text-primary theme token, and the DRY extraction into emptySessionCopy is the right call. Diff stayed surgical too — no scope creep, no formatting churn. Approving and merging now. Hope you'll send more — the dashboard has a few more good first issue tagged tickets if you want another swing. |
|
Thank you for the thoughtful review and for merging it. I appreciate the clear issue write-up — it made it easy to keep the change focused. I’ll keep an eye on the good first issue tickets and would be happy to take another swing where I can help. Glad to contribute. |
Description
Adds clearer empty-state guidance to the Sessions page when no session data exists. The non-search/non-filter empty state now keeps the existing “No sessions found” heading and adds guidance telling users to run
claudeto start a session.Search/filter empty states still show “No matching sessions” unchanged.
Closes #8
Type of Change
Checklist
npm test)npx tsc --noEmit)