Mission-planning: Enhance mission creation and editing workflow by ArturoManzoli · Pull Request #2541 · bluerobotics/cockpit

ArturoManzoli · 2026-03-24T17:17:57Z

Survey editing

Make all surveys in the mission history editable, not just the latest one;
Persist survey edits across boots and mode changes;

Survey operations

Add ability to swap a survey's entry and exit waypoints via the context menu;
Add ability to insert a survey between existing waypoints in the mission path;

Undo system

Improve undo functionality with dedicated survey polygon undo stack;
Fix spurious vertex creation when dragging a survey polygon to reposition it;

UI polish

Prevent the scan-direction dial from appearing off-screen when surveys are near viewport edges;
Add a reusable RadialMenu component for radial/pie-style context menus

Relevant to #2516
Close #2445
Close #2446
Close #1349
Close #2052

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

…changes Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com> Made-with: Cursor

ES-Alexander

Only had time for a brief test, and a skim through the code.

From that testing, things seem to be generally functioning as expected - nice! :-)

Some minor issues:

If the survey menu is left / up from the survey then it's appearing quite far away
In the top left corner, the survey menu can still end up partly out of the screen
Possibly out of scope, but the right click context menu for the map can also go off the screen at the bottom
The tiny arrows for showing the redo and mission load buttons seem hard to use / understand
- I think it makes more sense to move the hide/show mission estimates config into the mission settings, and add an icon to hide mission estimates on the actual mission estimates widget, and then use the extra space to just have the undo/redo | save/load | delete | settings menu options in full

ES-Alexander · 2026-03-25T16:56:01Z

+  if (isCreatingSurvey.value) return
+
+  const snapshot = missionStore.popRedoSnapshot()
+  if (!snapshot) return


Should there be some kind of warning here for the user, that the redo stack is empty?

Added the warning, even though this return was a second layer of protection. I don't think this will be triggered on a normal situation

ES-Alexander · 2026-03-25T17:01:53Z

-              </template>
-            </v-tooltip>
+            <div v-if="missionStore.currentPlanningWaypoints.length > 0" class="speed-dial-group">
+              <v-tooltip location="top" text="Undo (Ctrl+Z)">


Would be good if this could say Cmd+Z for Mac, or use the command symbol (⌘). Same for the redo case.

ArturoManzoli · 2026-03-26T12:12:52Z

Only had time for a brief test, and a skim through the code.

From that testing, things seem to be generally functioning as expected - nice! :-)

Some minor issues:

If the survey menu is left / up from the survey then it's appearing quite far away

Fixed. Now its under the pointer's edge unless too close to the edges, where it opens a little closer to the screen's center.

In the top left corner, the survey menu can still end up partly out of the screen

Fixed

Possibly out of scope, but the right click context menu for the map can also go off the screen at the bottom

It was actually on the scope. Fixed too

The tiny arrows for showing the redo and mission load buttons seem hard to use / understand

Indeed, that wasn't the best layout. Moved the mission estimates hide button to its container and added a switch to the mission options menu.

ES-Alexander

Noting that I haven't looked through the code super closely - I've mostly just tested functionality.
Approving because I don't expect these remaining fixes need re-testing.

With a turnaround distance specified, it's possible to undo into a state where just the turnaround lines are shown, during survey creation:
- This would ideally be cleared as part of the undo process, but it still clears fine when enough polygon points are drawn to generate new lines, or if the survey gets cancelled, so it's not a huge deal either way (just feels a bit janky).
When you swap the start and end points, then edit the survey polygon, then generate waypoints, the swapping is forgotten
- It seems intuitive that it would be maintained, but it's one extra button click after a potentially quite major change to the survey, so again just a small polish point.

ES-Alexander · 2026-03-26T20:57:15Z

+
+const performUndo = (): void => {
+  const snapshot = missionStore.popUndoSnapshot()
+  if (!snapshot) return


We should also have a warning about this, like the redo one.

ES-Alexander · 2026-03-26T20:58:32Z

+
+  const snapshot = missionStore.popRedoSnapshot()
+  if (!snapshot) {
+    openSnackbar({ variant: 'error', message: 'No more steps to redo.', duration: 2000 })


Is there a way to suppress this if the warning is already being shown? If I hold down Cmd+Y until the stack is empty then I get a big spam of warnings - one would be enough.

Added the message also to the undo command, and on both, added a trigger to display the message only once per cycle

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

ArturoManzoli · 2026-03-27T12:12:36Z

With a turnaround distance specified, it's possible to undo into a state where just the turnaround lines are shown, during survey creation:

Fixed!

When you swap the start and end points, then edit the survey polygon, then generate waypoints, the swapping is forgotten

Fixed!

ArturoManzoli added 5 commits March 24, 2026 14:42

mission-planning: make all survey history editable

f9d784a

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

mission-planning: make surveys edit persistant across boots and mode …

cd6c41d

…changes Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

mission-planning: add function to swap survey's entry and exit

547e042

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

components: radial-menu: add radial menu component

2a461b0

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com> Made-with: Cursor

mission-planning: add survey between waypoints

2d51c04

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com> Made-with: Cursor

ArturoManzoli force-pushed the 2445-fix-survey-polygons-ux branch from e27269f to 1434d91 Compare March 24, 2026 18:03

ArturoManzoli marked this pull request as ready for review March 24, 2026 18:03

ArturoManzoli requested review from ES-Alexander and rafaellehmkuhl March 24, 2026 18:03

ES-Alexander requested changes Mar 25, 2026

View reviewed changes

ArturoManzoli force-pushed the 2445-fix-survey-polygons-ux branch from 1434d91 to c93cbe2 Compare March 26, 2026 12:04

ArturoManzoli requested a review from ES-Alexander March 26, 2026 12:14

ES-Alexander approved these changes Mar 26, 2026

View reviewed changes

ES-Alexander added the docs-needed Change needs to be documented label Mar 27, 2026

ArturoManzoli added 2 commits March 27, 2026 09:10

mission-planning: improve undo function on mission planning

29c7f51

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

mission-planning: prevent scan-direction-dial from apperaring off-screen

6a3cabd

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>

ArturoManzoli force-pushed the 2445-fix-survey-polygons-ux branch from c93cbe2 to 6a3cabd Compare March 27, 2026 12:11

ArturoManzoli merged commit 52fb1a2 into bluerobotics:master Mar 27, 2026
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Mission-planning: Enhance mission creation and editing workflow#2541

Mission-planning: Enhance mission creation and editing workflow#2541
ArturoManzoli merged 7 commits into
bluerobotics:masterfrom
ArturoManzoli:2445-fix-survey-polygons-ux

ArturoManzoli commented Mar 24, 2026 •

edited

Loading

Uh oh!

ES-Alexander left a comment

Uh oh!

ES-Alexander Mar 25, 2026

Uh oh!

ArturoManzoli Mar 26, 2026

Uh oh!

ES-Alexander Mar 25, 2026

Uh oh!

ArturoManzoli Mar 26, 2026

Uh oh!

ArturoManzoli commented Mar 26, 2026

Uh oh!

ES-Alexander left a comment

Uh oh!

ES-Alexander Mar 26, 2026

Uh oh!

ES-Alexander Mar 26, 2026

Uh oh!

ArturoManzoli Mar 27, 2026

Uh oh!

ArturoManzoli commented Mar 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

ArturoManzoli commented Mar 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ES-Alexander left a comment

Choose a reason for hiding this comment

Uh oh!

ES-Alexander Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

ArturoManzoli Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

ES-Alexander Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

ArturoManzoli Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

ArturoManzoli commented Mar 26, 2026

Uh oh!

ES-Alexander left a comment

Choose a reason for hiding this comment

Uh oh!

ES-Alexander Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

ES-Alexander Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

ArturoManzoli Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

ArturoManzoli commented Mar 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ArturoManzoli commented Mar 24, 2026 •

edited

Loading