Skip to content

Android demo app improvements (bottom sheet, recenter button, etc.)#884

Merged
ianthetechie merged 5 commits into
stadiamaps:mainfrom
klemensz:pr-871-clean-demo-app-improvements
May 27, 2026
Merged

Android demo app improvements (bottom sheet, recenter button, etc.)#884
ianthetechie merged 5 commits into
stadiamaps:mainfrom
klemensz:pr-871-clean-demo-app-improvements

Conversation

@klemensz

Copy link
Copy Markdown
Contributor

Summary

This follow-up improves the Android demo app UX on top of the ongoing maplibre-compose migration in #853.

Instead of starting navigation immediately from a long press or search result, the demo now uses an explicit destination-selection flow with a preview bottom sheet, camera preview, and a browse-mode recenter button.

Why

This brings the demo app closer to a real navigation app and exercises more of the maplibre-compose integration in realistic user flows. It is split into a separate PR so the original migration PR does not grow even larger.

Changes

  • add destination selection state to the demo scene / view model
  • show a bottom sheet before starting navigation
  • preview the selected destination with camera updates and adjusted bottom padding
  • add a recenter button while not navigating
  • keep the dropped-pin overlay, now backed by typed FeatureCollection data instead of raw JSON strings
  • include a small MapControls padding fix for top system insets

Notes

This is primarily demo-app UX work. The only ui-maplibre change is the small MapControls inset adjustment.

Screenshots

image image

…o app based on maplibre-compose, featuring a bottom sheet for location details and improved camera handling.
@klemensz klemensz changed the title Demo app improvements (bottom sheet, recenter button, etc.) Android demo app improvements (bottom sheet, recenter button, etc.) Apr 28, 2026
klemensz added 3 commits May 20, 2026 21:03
# Conflicts:
#	android/demo-app/src/main/java/com/stadiamaps/ferrostar/DemoNavigationScene.kt
#	android/demo-app/src/main/java/com/stadiamaps/ferrostar/DemoNavigationViewModel.kt
#	android/demo-app/src/main/java/com/stadiamaps/ferrostar/NotNavigatingOverlay.kt
#	android/ui-maplibre/src/main/java/com/stadiamaps/ferrostar/maplibreui/runtime/MapControls.kt
implementation libs.androidx.compose.ui.graphics
implementation libs.androidx.compose.ui.tooling
implementation libs.androidx.compose.material3
implementation libs.androidx.compose.material.icon.extended

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can probably should replace this with SVGs (it's not a merge blocker for the demo app but does increase build times).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ianthetechie
ianthetechie merged commit ca5dff4 into stadiamaps:main May 27, 2026
14 checks passed
@klemensz
klemensz deleted the pr-871-clean-demo-app-improvements branch June 1, 2026 08:45
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.

3 participants