Deselect a point segmentation when a right-click confirms it

client/dive-common/use/useModeManager.spec.ts

@@ -761,7 +761,7 @@ describe('entering polygon editing', () => {
 });
 
 describe('confirming a point segmentation', () => {
-  it('leaves edit mode with the detection still selected, and removes one with nothing drawn', () => {
+  it('deselects the detection, and removes one with nothing drawn', () => {
     const recipe = new SegmentationPointClick();
     const { modeManager: manager, cameraStore } = makeHarness(undefined, [recipe]);
     recipe.activate();
@@ -770,9 +770,10 @@ describe('confirming a point segmentation', () => {
     manager.handler.trackEdit(drawn);
     recipe.resetPoints();
     manager.handler.confirmRecipe();
-    expect(manager.selectedTrackId.value).toBe(drawn);
+    expect(manager.selectedTrackId.value).toBeNull();
     expect(manager.editingTrack.value).toBe(false);
     expect(recipe.active.value).toBe(true);
+    expect(cameraStore.getPossibleTrack(drawn, 'left')).toBeDefined();
 
     const empty = manager.handler.trackAdd();
     recipe.resetPoints();
@@ -830,22 +831,20 @@ describe('a right-click that enters point segmentation editing', () => {
     manager.handler.confirmRecipe();
     expect(manager.selectedTrackId.value).toBe(first);
     expect(manager.editingTrack.value).toBe(true);
-    // A later right-click with no points placed leaves edit mode with the
-    // detection still selected, as the other annotation types do.
+    // A later right-click with no points placed finalizes and deselects it.
     press();
     manager.handler.confirmRecipe();
-    expect(manager.selectedTrackId.value).toBe(first);
+    expect(manager.selectedTrackId.value).toBeNull();
     expect(manager.editingTrack.value).toBe(false);
     // So does one after a reset by the user, even within the same press.
     press();
     manager.handler.trackEdit(second);
     recipe.resetPoints();
     expect(recipe.wasReset).toBe(true);
     manager.handler.confirmRecipe();
-    expect(manager.selectedTrackId.value).toBe(second);
+    expect(manager.selectedTrackId.value).toBeNull();
     expect(manager.editingTrack.value).toBe(false);
     // With nothing selected a right-click changes nothing.
-    manager.handler.trackSelect(null, false);
     press();
     manager.handler.confirmRecipe();
     expect(recipe.active.value).toBe(true);

client/dive-common/use/useModeManager.ts

@@ -1425,14 +1425,9 @@ export default function useModeManager({
     // Re-arm the recipe for the next detection first: activating it
     // re-selects the current track in edit mode (handleSetAnnotationState).
     activeSegRecipes.forEach((r) => r.activate());
-    // Then leave edit mode with the detection still selected, as the other
-    // annotation types do; one with nothing drawn is removed instead.
     const confirmedId = selectedTrackId.value;
-    if (confirmedId !== null && _removeIfEmpty(confirmedId)) {
-      selectTrack(null, false);
-    } else {
-      selectTrack(confirmedId, false);
-    }
+    if (confirmedId !== null) _removeIfEmpty(confirmedId);
+    selectTrack(null, false);
   }
 
   /**