Skip to content

Add kanban task creation and polish the task-creation UI#40

Merged
michaelzick merged 3 commits into
mainfrom
ui-fixes-kanban-task-creation
Jun 6, 2026
Merged

Add kanban task creation and polish the task-creation UI#40
michaelzick merged 3 commits into
mainfrom
ui-fixes-kanban-task-creation

Conversation

@michaelzick

Copy link
Copy Markdown
Owner

What & why

The board had no way to create tasks, and a UI audit found the new create-task panel diverged from the planner's equivalent form and rendered with a misaligned two-column layout. This branch adds board task creation and brings both task-creation forms into a consistent, polished, aligned state.

Changes

Feature — f871327 (Improve board planner links and add kanban task creation)

  • New board "New task" panel + create handler wired into the toolbar and page
  • Board ↔ planner links and status-move helpers (kanban-dnd, kanban-utils, kanban-create-task-utils)
  • Tests for the new flow (kanban-page, kanban-utils, app-shell)

UI polish — e6541f5 (Polish kanban task creation and unify task-form UI)

  • Reworked the board create-task panel into a balanced two-column layout: equal-height columns (notes stretches to fill), with Cancel/Add task in a full-width footer divider — this fixes the misaligned rows
  • Added a shared Select primitive with the same focus ring as Input; used in both board and planner create forms
  • Shared the Toggl project label + helper text between board and planner so both forms present identical guidance
  • Added a tokenized destructive button variant + --danger / --danger-strong / --danger-soft tokens (both themes); applied to the kanban delete action
  • Fixed the invisible light-mode drag-handle hover (now uses --panel-hover)
  • Corrected heading hierarchy: board h2, columns h3, card titles h4
  • Consistent duration preset labels (units on all)
  • Pointed the web dev launch config at port 6173 to match vite

Testing

  • pnpm lint and pnpm typecheck pass
  • Web tests: 142/142 relevant tests pass. One pre-existing timer flake in use-auto-google-task-sync.test.tsx (unrelated to this branch; passes in isolation, fails only under full-suite timer load)
  • The create-task panel was verified live in the browser across dark/light themes, narrow/wide breakpoints, and the title-required validation state

Reviewer notes

  • The source audit (ui-audit-kanban-task-creation.md) is intentionally not included in this PR.

🤖 Generated with Claude Code

michaelzick and others added 3 commits June 5, 2026 23:30
- Rework the board create-task panel into a balanced two-column layout: equal-height columns (notes stretches to fill), with Cancel/Add task in a full-width footer divider so rows no longer misalign
- Add a shared Select primitive with the same focus ring as Input; use it in both the board and planner create forms
- Share Toggl project label + helper text between board and planner so the forms present identical guidance
- Add a tokenized destructive button variant and --danger/--danger-strong/--danger-soft tokens (both themes); apply to the kanban delete action
- Fix the invisible light-mode drag-handle hover (use --panel-hover)
- Correct heading hierarchy: board h2, columns h3, card titles h4
- Use consistent duration preset labels (units on all)
- Point the web dev launch config at port 6173 to match vite

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
CI's Dependency Audit (pnpm audit --audit-level=moderate) was failing on 7 pre-existing transitive advisories unrelated to the UI work:

- hono <4.12.21 (via @modelcontextprotocol/sdk): bump the existing pnpm override ^4.12.18 -> ^4.12.21 (resolves to 4.12.23)
- react-router <7.15.0 (via react-router-dom): raise apps/web react-router-dom ^7.8.2 -> ^7.15.0 (resolves to 7.17.0), clearing the turbo-stream RCE, __manifest DoS, and protocol-relative open-redirect advisories

pnpm audit now reports no known vulnerabilities; typecheck, build, and all server/web tests pass.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@michaelzick michaelzick merged commit 765df4e into main Jun 6, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant