Skip to content

Improve map screen UX: toggle button, title fix, configurable data overlay - #4797

Open
cagnulein wants to merge 2 commits into
masterfrom
worktree-map-screen-ux-hr-power
Open

Improve map screen UX: toggle button, title fix, configurable data overlay#4797
cagnulein wants to merge 2 commits into
masterfrom
worktree-map-screen-ux-hr-power

Conversation

@cagnulein

@cagnulein cagnulein commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

Based on testing feedback from Maksim Savrilov (thread "HR zones tile and graph, disussion"):

  • The map icon in the toolbar now acts as a toggle: tap once to open the map, tap again to close it and go back to the tiles, instead of relying on the shared back/drawer button which was confusing to use as an exit from the map screen.
  • Fixed the toolbar title overlapping the map icon on narrow (phone) screens.
  • Removed the dead "Close" button in GoogleMap.qml — it was never actually wired to close the map (the popupclose signal had no listener), and would likely have been hidden behind the WebView layer on mobile anyway.
  • The resizable data box shown on the map screen now respects the same tile visibility switches already used by the main tiles grid (Settings → Tiles): tile_speed_enabled, tile_cadence_enabled, tile_heart_enabled, tile_calories_enabled, tile_odometer_enabled, tile_watt_enabled, tile_elapsed_enabled, tile_inclination_enabled, tile_resistance_enabled, tile_elevation_enabled (also used for the Altitude line, since there's no dedicated altitude tile). No new settings were introduced. The box itself was already resizable (drag the bottom-right handle) with auto-fitting font size.

Test plan

  • Open a GPX/live workout, tap the map icon: map opens; tap it again: map closes and returns to tiles.
  • Verify the toolbar title no longer overlaps the map icon on a phone-sized window.
  • In Settings → Tiles, disable a few tiles (e.g. keep only Watt and Heart Rate enabled) and confirm only those appear in the map's data box.
  • Confirm the data box is still resizable via the corner handle and the font still auto-fits.

🤖 Generated with Claude Code

cagnulein and others added 2 commits July 9, 2026 16:59
… data overlay

Reported by Maksim Savrilov via email testing feedback:
- The map icon now acts as a toggle (tap to open, tap again to close and
  return to tiles) instead of relying on the shared back/drawer button,
  which was confusing.
- Fixed the toolbar title overlapping the map icon on narrow screens.
- Removed the dead "Close" button in GoogleMap.qml (never wired to the
  toggle signal, and effectively unreachable behind the WebView layer).
- Added per-metric visibility settings for the resizable data box shown
  on the map screen (Speed, Cadence, HR, Calories, Odometer, Watt,
  Elapsed, Inclination, Resistance, Altitude, Elevation), so users can
  show only the metrics they care about instead of always seeing all 11.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Drop the new map_metric_*_enabled settings and wire the map's data box
visibility to the same tile_*_enabled switches already used by the main
tiles grid (Settings -> Tiles), avoiding a duplicate parallel settings
group for the same concept. There is no dedicated "altitude" tile, so
that line follows the elevation tile switch instead.

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

stale Bot commented Jul 24, 2026

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the wontfix This will not be worked on label Jul 24, 2026
@stale stale Bot closed this Aug 1, 2026
@cagnulein cagnulein reopened this Aug 1, 2026
@stale stale Bot removed the wontfix This will not be worked on label Aug 1, 2026
@stale

stale Bot commented Aug 16, 2026

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the wontfix This will not be worked on label Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant