fix: don't double-activate tools unnecessarily

saalfeldlab · Aug 14, 2024 · c16570c · c16570c
1 parent 2d77497
commit c16570c
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 17 deletions.
diff --git a/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/Modes.kt b/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/Modes.kt
@@ -31,9 +31,6 @@ import org.janelia.saalfeldlab.fx.util.InvokeOnJavaFXApplicationThread
 import org.janelia.saalfeldlab.paintera.PainteraBaseKeys
 import org.janelia.saalfeldlab.paintera.config.input.KeyAndMouseBindings
 import org.janelia.saalfeldlab.paintera.control.actions.AllowedActions
-import org.janelia.saalfeldlab.paintera.control.modes.NavigationTool.activeViewer
-import org.janelia.saalfeldlab.paintera.control.modes.NavigationTool.mode
-import org.janelia.saalfeldlab.paintera.control.modes.NavigationTool.removeFrom
 import org.janelia.saalfeldlab.paintera.control.tools.REQUIRES_ACTIVE_VIEWER
 import org.janelia.saalfeldlab.paintera.control.tools.Tool
 import org.janelia.saalfeldlab.paintera.control.tools.ToolBarItem
@@ -86,14 +83,15 @@ interface ToolMode : SourceMode {
 		}
 	}
 
-	fun switchTool(tool: Tool?) : Job? {
+	fun switchTool(tool: Tool?): Job? {
 		if (activeTool == tool)
 			return null
+
 		LOG.debug { "Switch from $activeTool to $tool" }
 
 		/* Deactivate off the main thread */
 		val deactivateJob = CoroutineScope(Dispatchers.Default).launch {
-			LOG.trace {"Deactivated $activeTool"}
+			LOG.trace { "Deactivated $activeTool" }
 			activeTool?.deactivate()
 		}
 
@@ -105,15 +103,15 @@ interface ToolMode : SourceMode {
 			showToolBars()
 			tool?.activate()
 			activeTool = tool
-			LOG.trace {"Activated $activeTool"}
+			LOG.trace { "Activated $activeTool" }
 		}
 
 		deactivateJob.invokeOnCompletion {
 			/* If the mode was changed before we can activate, don't activate anymore */
 			if (paintera.baseView.activeModeProperty.value == this@ToolMode)
 				activateJob.start()
 		}
-  		return activateJob
+		return activateJob
 	}
 
 	private fun showToolBars(show: Boolean = true) {

diff --git a/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/NavigationControlMode.kt b/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/NavigationControlMode.kt
@@ -71,10 +71,6 @@ object NavigationControlMode : AbstractToolMode() {
 
 	override val tools: ObservableList<Tool> = FXCollections.observableArrayList(NavigationTool)
 
-	override fun enter() {
-		super.enter()
-		switchTool(NavigationTool)
-	}
 }
 
 object NavigationTool : ViewerTool() {

diff --git a/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/RawSourceMode.kt b/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/RawSourceMode.kt
@@ -206,9 +206,6 @@ object RawSourceMode : AbstractToolMode() {
 			old?.viewer()?.removeActionSet(actionSet)
 			new?.viewer()?.installActionSet(actionSet)
 		}
-
-		/* set the currently activeTool for this viewer */
-		switchTool(activeTool ?: NavigationTool)
 	}
 
 	override fun enter() {

diff --git a/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/ViewLabelMode.kt b/src/main/kotlin/org/janelia/saalfeldlab/paintera/control/modes/ViewLabelMode.kt
@@ -27,9 +27,6 @@ object ViewLabelMode : AbstractToolMode() {
 			old?.viewer()?.removeActionSet(actionSet)
 			new?.viewer()?.installActionSet(actionSet)
 		}
-
-		/* set the currently activeTool for this viewer */
-		switchTool(activeTool ?: NavigationTool)
 	}
 
 	override fun enter() {