ginnoir/famapp
1
0
Fork 0
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

Quick-add opens create UI, not module page #1

New Issue
Closed
opened 2026-07-03 23:54:47 -05:00 by ginnoir · 0 comments
ginnoir commented 2026-07-03 23:54:47 -05:00
Owner
Copy Link
Copy Source

Quick-add actions must open create dialogs/sheets, not only navigate to module pages.

Background / current behavior

Every quick-add entry (FAB sheet and cmd-k palette) currently navigates to its module page (e.g. "New event" → /calendar). Task 21 intentionally used url navigation as a placeholder; inline create was deferred.

Relevant code: src/modules/_core/module.ts (QuickAddAction), module manifests (quickAdds), QuickAddSheet / CommandPalette under src/components/.

Expected behavior

Each quick-add opens the creation UI for that entity (dialog/sheet) from wherever the button lives, without requiring a full page navigation first. Navigation-only is not acceptable.

Acceptance criteria

  • "New event" opens the event create dialog/sheet with create controls visible.
  • List/note/garden (and other registered) quick-adds open their create UI the same way.
  • Cmd-k and FAB sheet behave identically for the same action.
  • Playwright: extend tests/e2e/dashboard.spec.ts and/or add tests/e2e/quick-add.spec.ts — open FAB → action opens dialog/sheet, not only a route change.

Priority

P1 (batch step 1). Task brief: docs/tasks/80-quick-add-create-ui.md.

<brief>Quick-add actions must open create dialogs/sheets, not only navigate to module pages.</brief> ## Background / current behavior Every quick-add entry (FAB sheet and cmd-k palette) currently navigates to its module page (e.g. "New event" → `/calendar`). Task 21 intentionally used `url` navigation as a placeholder; inline create was deferred. Relevant code: `src/modules/_core/module.ts` (`QuickAddAction`), module manifests (`quickAdds`), `QuickAddSheet` / `CommandPalette` under `src/components/`. ## Expected behavior Each quick-add opens the **creation UI** for that entity (dialog/sheet) from wherever the button lives, without requiring a full page navigation first. Navigation-only is not acceptable. ## Acceptance criteria - [ ] "New event" opens the event create dialog/sheet with create controls visible. - [ ] List/note/garden (and other registered) quick-adds open their create UI the same way. - [ ] Cmd-k and FAB sheet behave identically for the same action. - [ ] Playwright: extend `tests/e2e/dashboard.spec.ts` and/or add `tests/e2e/quick-add.spec.ts` — open FAB → action opens dialog/sheet, not only a route change. ## Priority P1 (batch step 1). Task brief: `docs/tasks/80-quick-add-create-ui.md`.
ginnoir added the bugpriority/p1 labels 2026-07-03 23:54:47 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is broken
 enhancement
Improvement to existing behavior
 epic
Large multi-issue effort
 feature
New user-facing capability
 priority/p1
Next batch
 priority/p2
After P1
 priority/p3
Backlog
 research
Investigation / ADR only
No labels bug priority/p1
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
ginnoir (Ginnoir)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ginnoir/famapp#1