Repository navigation
feat: add agent authoring, desktop pairing, and captuto v2 redesign - #109
Merged
Merged
Conversation
Replace the top-bar dashboard layout with a sidebar layout (search, status nav with counts, list/grid views, URL-driven filters shared between the sidebar and page), and reskin the editor's top bar from a dark theme to the light theme already used by the rest of the editor. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vo583N1CQqRRdTWA7ghXE6
…-075b # Conflicts: # apps/web/app/dashboard/layout.tsx # apps/web/app/dashboard/page.tsx # apps/web/components/dashboard/DashboardTopBar.tsx # apps/web/lib/dashboard/query.ts
Address review-code-dev findings on the v2 redesign: - add a mobile/tablet top bar (search, tab menu, download/settings/ sign-out) since the desktop sidebar is hidden below the lg breakpoint - clear the React Query cache on sign-out again (dropped when DashboardTopBar was replaced), preventing stale cached guide data from leaking into the next signed-in session on a shared browser - guard the editor title field's prop-sync effect so an in-flight title mutation (autosave rollback, AI clean-up) can't silently overwrite an in-progress keystroke - restore role="group"/aria-pressed on the dashboard's tab filter - relabel the header CTA from "New guide" to "Connect your agent" since it links to Settings, not a guide-creation flow Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vo583N1CQqRRdTWA7ghXE6
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vo583N1CQqRRdTWA7ghXE6
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
lgbreakpoint.Screenshots
Verification
pnpm exec tsc --noEmit(apps/web) - passedpnpm exec eslinton all changed files - passed, no warningspnpm test:run(full monorepo suite via turbo, includes web + extension) - 39 files / 436 tests passedagent-browser- logged in, exercised dashboard list/grid views, tab filters, search, empty states, mobile nav (sidebar hidden belowlg, mobile bar takes over), and the editor's Edit/Preview modes and title fieldReview Gate
review-code-dev(standard effort, frontend lens): found 2 P1 + 3 P2 issues on the new redesign commit — all fixed and re-verified:hidden lg:flexwith no fallback) — added a mobile top bar with search + menurole="group"/aria-pressed— restoredplans/review-code-dev/runs/20260909-082015-worktree-green-forest-075b/(local, gitignored)Risk
Human Review Notes
🤖 Generated with Claude Code
https://claude.ai/code/session_01Vo583N1CQqRRdTWA7ghXE6