[dashboard] Unify workspace (instance) visuals across dashboard
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 13.8k
- Forks
- 1.4k
- Avg merge
- 17h 58m
- Merged PRs (30d)
- 5
Description
At the moment we copy'n'paste code for displaying a workspace (instance) and their state in various places, leading to incosistencies.
We already have code in place for the status indicator, for instance, but don't use it everywhere.
Ideally, the different parts (status indicator, unpushed changes, etc.) would be implemented in recombinable functions, so we can still vary the resulting look.
regarding unpushed changes: If there are no changes, we can probably skip that element instead of displaying "No changes".
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with components/dashboard/src/workspaces/WorkspaceEntry.tsx, especially the existing status indicator referenced in the issue, and trace the other dashboard locations that display workspace state. Refactor the repeated visual parts into recombinable functions, use them consistently, and omit the unpushed-changes element when there are no changes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100