Project view: entity disambiguation, roomier spacing, dashboard landing - #109
Conversation
Refine the project view restyle: - Shape language (mirrors the TUI): worktree = thick-outline square, agent = filled circle, service = diamond + a "service" tag. Fill/outline color still encodes status (green = running). Adds StatusDotMini shape/outline + TypeTag. - Nesting: agents/services hang off a vertical guide line indented under their worktree, in both the sidebar and the page. - Page view adopts the sidebar's flat list look (no table borders / header band), scaled ~20% roomier (padding, type, row height) for a GUI surface; sidebar bumped to match while staying a denser nav rail. - Extract the worktree dashboard into a shared WorktreeDashboard component. - Make the Project screen's "Dashboard" the first pill and default section, and route every landing (root URL, default project, project switch) to /project's Dashboard section. The legacy standalone dashboard route redirects there. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe PR converts a standalone dashboard route into a redirect endpoint and establishes a new project-scoped "dashboard" section. A new ChangesDashboard Route Migration and Component Refactor
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
app/app/(main)/(tabs)/(dashboard)/index.tsxESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox. app/app/(main)/(tabs)/project/index.tsxESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox. app/components/ProjectSidebar.tsxESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox.
Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
…redirect The standalone dashboard route now <Redirect>s to the Project screen's Dashboard section, so the component is rendered only there; drop the stale 'dashboard route' references in the header/padded comments. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Review note (in-runtime): CodeRabbit reported its org is out of review credits / rate-limited (its summary comment shows "Review limit reached"), so its
No correctness, routing, redirect-loop, state-handling, key-collision, hook-dependency, or accessibility issues found. |
What
Follow-up polish on the project-view restyle (builds on #107).
Entity disambiguation (shape language, mirrors the TUI)
servicetag. Fill/outline color still encodes status (green = running, muted = idle, faint = empty).StatusDotMinigainsshape/outline; newTypeTag.Spacing
Dashboard as the canonical landing
components/WorktreeDashboard.tsx./project's Dashboard section. The legacy standalone/dashboard route now<Redirect>s there. The sidebar Dashboard/Views tree toggle stays separate.Behavior
Navigation, selection, and service actions are preserved. New interactions are additive (collapsible groups, empties disclosure, project-switch lands on /project dashboard).
Verification
yarn typecheck+ lint clean on all changed files (pre-existingcli-auth.tsxlint error untouched). Validated locally in LOCAL_MODE against aimux-dev with live agents/services across desktop widths; per-page mobile polish deferred.🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
UI/UX Updates