From 5b56d430e9ef81f35f25338b2b75805a5cf21314 Mon Sep 17 00:00:00 2001 From: scotttong Date: Fri, 4 Sep 2026 16:15:21 -0700 Subject: [PATCH] feat(ui): refine core navigation and task detail (#12854) ## Thinking Path > - Paperclip is the open source app people use to manage AI agents for work. > - The main navigation and task detail view are core operator surfaces. > - Several controls used different hover states, popover layouts, and spacing rules. > - Recent task actions also needed a compact menu and correct inbox archive behavior. > - These differences made the interface feel inconsistent and caused some content to look crowded or clipped. > - This pull request aligns these surfaces with the Paperclip design tokens and current interaction patterns. > - The benefit is a simpler and more consistent operator experience in light and dark modes. ## Linked Issues or Issue Description **What happened?** Profile and organization popovers used inconsistent layouts. Navigation controls used different hover and selected backgrounds. Task warnings and the composer could crowd nearby content. Archiving a recent task could also remove it from more than the inbox. **Expected behavior** Popover menus should use the same compact visual language. Navigation controls should share readable hover and selected tokens. Task detail content should keep consistent spacing. Archiving should hide a task from the inbox while keeping it in the task list. **Steps to reproduce** 1. Open the main sidebar in light or dark mode. 2. Open the profile and organization menus. 3. Hover navigation items, the organization trigger, the profile trigger, and the feedback flag. 4. Open a task with a warning banner and a long thread. 5. Use the recent task overflow menu and archive a task. **Paperclip version or commit** Reproduced on `master` before this branch. **Deployment mode** Local dev (`pnpm dev`). ## What Changed - Rebuilt the profile and organization popovers with compact token-based layouts. - Matched organization popover width and alignment to the profile popover. - Unified sidebar hover and selected states in light and dark modes. - Added a recent task overflow menu with rename, archive, and pause or restart actions. - Kept archived tasks in the task list while removing them from the inbox. - Improved warning banner and composer spacing in task detail views. - Added and updated focused UI tests for the changed behavior. ## Verification - `pnpm check:token-gates` passed. - `pnpm --filter @paperclipai/ui typecheck` passed. - The seven affected UI test files passed with 216 tests. - `pnpm --filter @paperclipai/ui build` passed. - GitHub CI passed the full build, typecheck and release registry, general test, serialized server, canary dry-run, and end-to-end matrices. - Greptile reviewed commit `6e296be85` at 5/5 with no outstanding actionable findings. ## Risks - Risk is limited to sidebar presentation, recent task actions, and task detail layout. - The recent task archive action now follows inbox-only archive semantics. - No database schema or public API contract changed. > I checked [`ROADMAP.md`](ROADMAP.md). This pull request does not duplicate planned core work. ## Model Used - OpenAI Codex, GPT-5.6. The model used high reasoning, tool use, and code execution. The context window size was not exposed. ## Checklist - [x] I have included a thinking path that traces from project context to this change - [x] I have specified the model used (with version and capability details) - [x] I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work - [x] I have searched GitHub for duplicate or related PRs and linked them above - [x] I have either (a) linked existing issues with `Fixes: #` / `Closes #` / `Refs #` OR (b) described the issue in-PR following the relevant issue template - [x] I have not referenced internal/instance-local Paperclip issues or links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip` URLs) - [x] My branch name describes the change (e.g. `docs/...`, `fix/...`) and contains no internal Paperclip ticket id or instance-derived details - [x] I have run tests locally and they pass - [x] I have added or updated tests where applicable - [x] I have updated relevant documentation to reflect my changes - [x] I have considered and documented any risks above - [x] All Paperclip CI gates are green - [x] Greptile is 5/5 with no open P2s, recommendations, or follow-ups - [x] I will address all Greptile and reviewer comments before requesting merge --------- Co-authored-by: Scott Tong Co-authored-by: Paperclip --- ui/src/components/Sidebar.tsx | 3 +- .../SidebarAccountMenu.production.tsx | 2 +- ui/src/components/SidebarAccountMenu.test.tsx | 24 +- ui/src/components/SidebarAccountMenu.tsx | 152 ++++----- ui/src/components/SidebarAgents.tsx | 2 +- .../SidebarCompanyMenu.production.tsx | 170 ++++++---- ui/src/components/SidebarCompanyMenu.test.tsx | 48 +-- ui/src/components/SidebarCompanyMenu.tsx | 166 ++++++---- ui/src/components/SidebarNavItem.test.tsx | 14 +- ui/src/components/SidebarNavItem.tsx | 4 +- ui/src/components/SidebarProjects.tsx | 4 +- ui/src/components/SidebarRecentTasks.test.tsx | 282 +++++++++++++++- ui/src/components/SidebarRecentTasks.tsx | 313 +++++++++++++++++- ui/src/components/SidebarStarredProjects.tsx | 4 +- ui/src/components/TaskChatThread.test.tsx | 9 +- ui/src/components/TaskChatThread.tsx | 9 +- ui/src/components/ThemeToggle.test.tsx | 17 + ui/src/components/ThemeToggle.tsx | 26 +- ui/src/index.css | 21 +- ui/src/pages/DesignGuide.tsx | 11 + ui/src/pages/IssueDetail.test.tsx | 12 + ui/src/pages/IssueDetail.tsx | 9 +- 22 files changed, 1009 insertions(+), 293 deletions(-) diff --git a/ui/src/components/Sidebar.tsx b/ui/src/components/Sidebar.tsx index 2c750c36ee..9276fbcda4 100644 --- a/ui/src/components/Sidebar.tsx +++ b/ui/src/components/Sidebar.tsx @@ -141,8 +141,7 @@ export function Sidebar() { data-slot="icon-button" aria-label={rail ? "New Task" : undefined} className={cn( - "flex items-center gap-2.5 mx-2 rounded-lg px-2 py-1.5 pointer-coarse:py-1 text-(length:--text-compact) font-medium text-foreground/80 hover:text-foreground transition-colors", - streamlinedUiEnabled ? "hover:bg-background" : "hover:bg-accent/50", + "flex items-center gap-2.5 mx-2 rounded-lg px-2 py-1.5 pointer-coarse:py-1 text-(length:--text-compact) font-medium text-foreground/80 transition-colors hover:bg-sidebar-accent hover:text-sidebar-accent-foreground", )} > diff --git a/ui/src/components/SidebarAccountMenu.production.tsx b/ui/src/components/SidebarAccountMenu.production.tsx index c136d4ea2f..738fcc7a62 100644 --- a/ui/src/components/SidebarAccountMenu.production.tsx +++ b/ui/src/components/SidebarAccountMenu.production.tsx @@ -235,7 +235,7 @@ export function SidebarAccountMenu({ target="_blank" rel="noreferrer" aria-label="Share feedback" - className="flex size-8 shrink-0 items-center justify-center rounded-lg text-foreground/80 transition-colors hover:bg-accent/50 hover:text-foreground focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring" + className="flex size-8 shrink-0 items-center justify-center rounded-lg text-muted-foreground/50 transition-colors hover:bg-accent/50 hover:text-foreground focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring" >