Skip to content

Add distance measurement persistence to maps#1171

Merged
1Blademaster merged 1 commit into
mainfrom
1169-feature-keep-distance-measurements-on-the-map
Apr 19, 2026
Merged

Add distance measurement persistence to maps#1171
1Blademaster merged 1 commit into
mainfrom
1169-feature-keep-distance-measurements-on-the-map

Conversation

@1Blademaster

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 19, 2026 17:50
@1Blademaster 1Blademaster linked an issue Apr 19, 2026 that may be closed by this pull request

Copilot AI left a comment

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.

Pull request overview

This PR moves distance measurement state out of component-local state and into Redux so measurements persist on both the dashboard map and missions map while navigating/closing context menus, and adds UI support for deleting a saved measurement via context menu.

Changes:

  • Add distanceMeasurements (draft start + saved items) to both missionSlice and dashboardSlice, with actions/selectors to manage them.
  • Update the dashboard and missions map components to use Redux-backed distance measurement state and to allow deleting a saved measurement.
  • Refactor DistanceMeasurementMarkers to render persistent start/end pins, a connecting line, and a midpoint label showing the measured distance.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
gcs/src/redux/slices/missionSlice.js Adds Redux state/actions/selectors for mission-map distance measurement persistence.
gcs/src/redux/slices/dashboardSlice.js Adds Redux state/actions/selectors for dashboard-map distance measurement persistence.
gcs/src/components/missions/missionsMap.jsx Switches distance measurement from local state to Redux and adds “Delete distance measurement” context action.
gcs/src/components/mapComponents/distanceMeasurement.jsx Reworks rendering to show persistent measurement pins/lines and a midpoint distance label.
gcs/src/components/dashboard/map.jsx Switches distance measurement from local state to Redux and adds “Delete distance measurement” context action.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@1Blademaster
1Blademaster merged commit e5f09b6 into main Apr 19, 2026
9 checks passed
@1Blademaster
1Blademaster deleted the 1169-feature-keep-distance-measurements-on-the-map branch April 19, 2026 17:55
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.

[FEATURE] Keep distance measurements on the map

2 participants