Repository navigation
feat(navigation): unify and refine transit guidance UI - #802
Open
Medformatik wants to merge 3 commits into
Open
Medformatik wants to merge 3 commits into
Medformatik wants to merge 3 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Transit guidance now uses one neutral journey sheet with a compact route strip and a pinned End/arrival footer. Boarding details, next-stop guidance, transfers and notices live in the same surface. Walking keeps the shared maneuver banner. Native Google Maps exploration informed the structure while OpenMapX keeps its theme and explicit boarding/alighting actions.
Outer surfaces share 28px corners, actions use pills, and inner rows use dividers. The expanded timeline folds intermediate and previous stops while keeping boarding, next, alighting and disrupted stops visible. Uncertain progress stays explicit. Browser speech and alarms have one session-level owner so expanding details cannot replay cues. Collapsed sizing accounts for docked map credits on small screens and switches to a compact route strip when the summary cannot fit. The shared maneuver banner uses the same outer radius in walking and driving.
Before / after
Same controlled public Aachen fixture and viewport, rendered with the production navigation view and map. Navigation processing was paused to hold the comparison state.
Walking, dark mode, transfer, small screen and desktop
Compact walking view when long text cannot fit below the maneuver:
Shared maneuver banner, also used by driving:
Verification
124 focused tests passed. Web, service-worker and integration type checks, translation parity and changed-file Biome passed. Inspected light/dark, walking, boarding, riding, alighting, transfer, cancellation, German, safe-area, short-screen and desktop renders. Exercised keyboard disclosure focus and scrolling to Settings with the footer pinned. Claude reviewed the design and implementation.
Google Maps was explored on Android with public simulated GPS. OpenMapX APK/device verification and the full suite were left unperformed. Full-suite execution is left to PR CI. No configuration or data migration.