tfks/trippin
tfks/trippin
4
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

Move trip actions to map toolbar #13

New issue
Closed
opened 2026-05-21 15:12:08 +00:00 by erik · 3 comments
erik commented 2026-05-21 15:12:08 +00:00
Owner
Copy link
  • Remove the trip card from the poi panel.
  • Add trip details to the top of poi panel (where trip card was): description, happened, tags
  • All actions (eg Edit, Upload Photos, ...) should be be added to the map toolbar as icons only.
  • Make the current Info action in the toolbar icon only as well.
  • Make the toolbar use flex-warp + justify-between to support long trip names on mobile.
- Remove the trip card from the poi panel. - Add trip details to the top of poi panel (where trip card was): description, happened, tags - All actions (eg Edit, Upload Photos, ...) should be be added to the map toolbar as icons only. - Make the current Info action in the toolbar icon only as well. - Make the toolbar use flex-warp + justify-between to support long trip names on mobile.
agent-dev commented 2026-05-21 15:18:23 +00:00
Member
Copy link

Ledger for: agent-dev

Status: review @ 2026-05-21T15:22:58.805Z

Tasks

  • Remove trip-card from poi-drawer render — replaced with _renderTripInfo() method showing date, tags, description
  • Add public methods on PoiDrawer: openUploadDialog(), openAddRouteDialog(), openDeleteDialog()
  • Add action buttons (Edit link, Upload, Route, Delete) to map toolbar in TripDetailPage.tsx — icon-only, owner-gated
  • Make Info button icon-only (removed <span class="hidden sm:inline">Info</span>)
  • Change toolbar to flex-wrap justify-between layout
  • Pass tags field in trip JSON to poi-drawer
  • Wire toolbar button clicks in TripMapCoordinator to poi-drawer public methods
  • Commit 12a565b, PR #14 opened

Evidence

  • Commit: 12a565b on branch chore/trip-toolbar-actions
  • PR: #14
  • just test::format test::rules test::types all pass (coverage skipped — pre-existing .env.test absent)
<!-- agent-dev:ledger --> ## Ledger for: agent-dev Status: review @ 2026-05-21T15:22:58.805Z --- ### Tasks - [x] Remove trip-card from poi-drawer render — replaced with `_renderTripInfo()` method showing date, tags, description - [x] Add public methods on PoiDrawer: `openUploadDialog()`, `openAddRouteDialog()`, `openDeleteDialog()` - [x] Add action buttons (Edit link, Upload, Route, Delete) to map toolbar in TripDetailPage.tsx — icon-only, owner-gated - [x] Make Info button icon-only (removed `<span class="hidden sm:inline">Info</span>`) - [x] Change toolbar to `flex-wrap justify-between` layout - [x] Pass `tags` field in trip JSON to poi-drawer - [x] Wire toolbar button clicks in TripMapCoordinator to poi-drawer public methods - [x] Commit 12a565b, PR #14 opened ### Evidence - Commit: 12a565b on branch chore/trip-toolbar-actions - PR: https://git.tfks.net/tfks/trippin/pulls/14 - `just test::format test::rules test::types` all pass (coverage skipped — pre-existing .env.test absent)
agent-dev removed their assignment 2026-05-21 15:23:04 +00:00
erik removed their assignment 2026-05-21 15:53:20 +00:00
agent-pm removed their assignment 2026-05-22 14:26:13 +00:00
agent-pm commented 2026-05-22 14:26:14 +00:00
Member
Copy link

Brief — QA

Phase: review. Run fjx qa prompt default for the cycle protocol.

<!-- pm:brief:qa --> # Brief — QA **Phase:** review. Run `fjx qa prompt default` for the cycle protocol.
agent-pm commented 2026-05-22 14:26:15 +00:00
Member
Copy link

Ledger for: agent-pm

Status: closed @ 2026-05-22T14:26:12.402Z
Briefs:

  • dev: absent
  • qa
  • build: absent

Activity

  1. i13/closed : applied

(PM manages this ledger; agent section unused)

<!-- agent-pm:ledger --> ## Ledger for: agent-pm Status: closed @ 2026-05-22T14:26:12.402Z Briefs: - dev: absent - [qa](#issuecomment-1495) - build: absent ### Activity 1. i13/closed : applied --- _(PM manages this ledger; agent section unused)_
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
No results found.
Labels
Clear labels
  
agent
blocked
 Needs human intervention

  
agent
new
 Waiting triage by PM

  
agent
review
 Awaiting next phase

  
agent
working
 Agent is actively implementing

  
complexity
high
 High complexity - use more capable LLM

  
complexity
low
 Low complexity - use simpler LLM

  
priority
high
 High priority

  
priority
low
 Low priority

  
priority
medium
 Medium priority

  
risk
high
 High risk — strict QA, no auto-release

  
risk
low
 Low risk — standard flow

  
risk
medium
 Medium risk — standard QA

  
type
bug
 Bug fix

  
type
chore
 Maintenance, deps, tooling

  
type
feature
 New feature — spec required

  
type
security
 Security change — spec + strict QA required

No labels
agent
blocked
agent
new
agent
review
agent
working
complexity
high
complexity
low
priority
high
priority
low
priority
medium
risk
high
risk
low
risk
medium
type
bug
type
chore
type
feature
type
security
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
agent-qa
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
tfks/trippin#13
No description provided.