Handle Tool::Finished returned by processKeyEvent (#1257) (#1264)

(cherry picked from commit 37cf805)

Co-authored-by: Patrick Roncagliolo <[email protected]>

Author: mergify[bot]

rviz_common/src/rviz_common/tool_manager.cpp

@@ -157,6 +157,7 @@ void ToolManager::handleChar(QKeyEvent * event, RenderPanel * panel)
     tool = shortkey_to_tool_map_[event->key()];
   }
 
+  int flags = 0;
   if (tool) {
     // if there is a incoming tool check if it matches the current tool
     if (current_tool_ == tool) {
@@ -166,7 +167,7 @@ void ToolManager::handleChar(QKeyEvent * event, RenderPanel * panel)
       // if no, check if the current tool accesses all key events
       if (current_tool_->accessAllKeys()) {
         // if yes, pass the key
-        current_tool_->processKeyEvent(event, panel);
+        flags = current_tool_->processKeyEvent(event, panel);
       } else {
         // if no, switch the tool
         setCurrentTool(tool);
@@ -175,7 +176,11 @@ void ToolManager::handleChar(QKeyEvent * event, RenderPanel * panel)
   } else {
     // if the incoming key triggers no other tool,
     // just hand down the key event
-    current_tool_->processKeyEvent(event, panel);
+    flags = current_tool_->processKeyEvent(event, panel);
+  }
+
+  if (flags & Tool::Finished) {
+    setCurrentTool(getDefaultTool());
   }
 }