chore: move trip actions to map toolbar #14

Merged

erik merged 3 commits from chore/trip-toolbar-actions into main

2026-05-22 14:24:08 +00:00

agent-dev commented

2026-05-21 15:22:41 +00:00

Member

Refs: #13

Summary

Removed the <trip-card> from the POI panel drawer; replaced with a lightweight trip info section showing the happened date, tags (as <sl-tag> chips), and description
Added icon-only action buttons (Edit, Upload Photos, Add/Update Route, Delete trip) to the map toolbar — visible to owners only
Made the existing Info button icon-only (no text on any breakpoint)
Changed toolbar layout to flex-wrap justify-between so long trip names wrap gracefully on mobile instead of overflowing

Test plan

Open a trip as owner: confirm Edit, Upload, Route, Delete, Info buttons appear in toolbar as icons only
Click Upload Photos button → upload dialog opens
Click Add/Update Route button → route dialog opens
Click Delete button → delete trip confirmation dialog opens
Click Edit icon → navigates to /trips/{id}/edit
Click Info button → POI panel drawer opens/closes
Open a trip as non-owner: confirm only Info button is visible
On mobile with a long trip name: confirm title wraps and action buttons stay on a second row without overflowing
POI panel header shows date, tags, description instead of trip card

🤖 Generated with Claude Code

Refs: #13 --- ## Summary - Removed the `<trip-card>` from the POI panel drawer; replaced with a lightweight trip info section showing the happened date, tags (as `<sl-tag>` chips), and description - Added icon-only action buttons (Edit, Upload Photos, Add/Update Route, Delete trip) to the map toolbar — visible to owners only - Made the existing Info button icon-only (no text on any breakpoint) - Changed toolbar layout to `flex-wrap justify-between` so long trip names wrap gracefully on mobile instead of overflowing ## Test plan - [ ] Open a trip as owner: confirm Edit, Upload, Route, Delete, Info buttons appear in toolbar as icons only - [ ] Click Upload Photos button → upload dialog opens - [ ] Click Add/Update Route button → route dialog opens - [ ] Click Delete button → delete trip confirmation dialog opens - [ ] Click Edit icon → navigates to `/trips/{id}/edit` - [ ] Click Info button → POI panel drawer opens/closes - [ ] Open a trip as non-owner: confirm only Info button is visible - [ ] On mobile with a long trip name: confirm title wraps and action buttons stay on a second row without overflowing - [ ] POI panel header shows date, tags, description instead of trip card 🤖 Generated with [Claude Code](https://claude.com/claude-code)

agent-dev added 1 commit

2026-05-21 15:22:41 +00:00

chore: move trip actions to map toolbar, replace trip card with info section in POI panel

Validate / validate (pull_request) Failing after 9s

Details

12a565b86f

- Remove trip-card from poi-drawer; replace with a lean trip info section
  showing happened date, tags (sl-tag chips), and description
- Add icon-only action buttons (Edit, Upload Photos, Add Route, Delete) to
  the map toolbar for owners; Info button is now icon-only too
- Toolbar uses flex-wrap + justify-between so long trip names wrap cleanly
  on mobile without pushing buttons off-screen
- Expose openUploadDialog / openAddRouteDialog / openDeleteDialog public
  methods on PoiDrawer; TripMapCoordinator wires toolbar button clicks to
  call them

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

~~agent-dev referenced this pull request 2026-05-21 15:23:00 +00:00~~

Thumbnails #2

erik referenced this pull request

2026-05-21 15:52:52 +00:00

Move trip actions to map toolbar #13

erik added 2 commits

2026-05-22 14:23:21 +00:00

Merge remote-tracking branch 'origin/main' into chore/trip-toolbar-actions dd9e0b4f01

chore: replace toolbar buttons with sl-button, add Escape to close POI panel

Validate / validate (pull_request) Failing after 5s

Details

93664d16f8

- Convert all map toolbar buttons/links to sl-button (size=small); edit link uses href prop. Delete button uses variant=danger outline.
- Add title to sl-button type declaration.
- Add keydown handler: Escape closes POI panel when no sl-dialog is open.
- Remove stale if-infoPanel guard around keydown cleanup.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

erik merged commit 439b8f4196 into main

2026-05-22 14:24:08 +00:00

erik deleted branch chore/trip-toolbar-actions

2026-05-22 14:24:08 +00:00

erik referenced this pull request from a commit

2026-05-22 14:24:09 +00:00

Merge pull request 'chore: move trip actions to map toolbar' (#14) from chore/trip-toolbar-actions into main