Skip to content

Confirm joining trips in a dialog - #532

Merged
joszz merged 2 commits into
mainfrom
fix/trip-join-dialog
Oct 23, 2026
Merged

joszz merged 2 commits into
mainfrom
fix/trip-join-dialog

Conversation

@joszz

@joszz joszz commented Oct 23, 2026

Copy link
Copy Markdown
Owner

On phones the armed Tap again to join label made the trip card's map, hide and join buttons wrap onto a row of their own.

  • The join button on a trip now only asks; it keeps its icon-only look on phones.
  • The trip log page shows one confirmation dialog (Cancel / Join trips) with the existing "cannot be undone" explanation. Both buttons hold while the join is saving.
  • joinNextConfirm now reads "Join trips" / "Ritten samenvoegen"; the --kept label modifier is gone.
  • Tests updated: the item emits on one tap, the view joins only after confirming and does nothing on Cancel.

🤖 Generated with Claude Code

joszz and others added 2 commits October 23, 2026 08:17
On phones the armed "Tap again to join" label pushed the trip's buttons
onto a row of their own. The join button now only asks, and the trip log
page confirms in a dialog that says the join cannot be undone.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The address lookup retries in rounds while the mocked API returns no
addresses. Pages left mounted kept asking in later tests, which made
"asks for no addresses while place names are switched off" flaky in CI.
Unmounting stops the lookup, as it does when the page closes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@joszz
joszz enabled auto-merge October 23, 2026 06:23
@joszz
joszz merged commit bb5f49b into main Oct 23, 2026
21 checks passed
@joszz
joszz deleted the fix/trip-join-dialog branch October 23, 2026 06:24
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