Improve node selection ~x3 removing unneded updateNow() called in non necesary places. Also ensure canvas is always redraw

1 files changed, 6 insertions, 4 deletions

diff --git a/src/ui/tools/node-tool.cpp b/src/ui/tools/node-tool.cpp
index 63161b3cd..937c1a657 100644
--- a/src/ui/tools/node-tool.cpp
+++ b/src/ui/tools/node-tool.cpp
@@ -458,7 +458,8 @@ void NodeTool::selection_changed(Inkscape::Selection *sel) {
 
     this->_multipath->setItems(shapes);
     this->update_tip(nullptr);
-    this->desktop->updateNow();
+    // This not need to be called canvas is updated on selection change on setItems
+    // this->desktop->updateNow();
 }
 
 bool NodeTool::root_handler(GdkEvent* event) {
@@ -548,7 +549,7 @@ bool NodeTool::root_handler(GdkEvent* event) {
                 SP_STROKE_LINEJOIN_MITER, SP_STROKE_LINECAP_BUTT);
 
             sp_canvas_bpath_set_fill(SP_CANVAS_BPATH(flash), 0, SP_WIND_RULE_NONZERO);
-
+            desktop->canvas->forceFullRedrawAfterInterruptions(5);
             this->flash_tempitem = desktop->add_temporary_canvasitem(flash,
                 prefs->getInt("/tools/nodes/pathflash_timeout", 500));
 
@@ -631,6 +632,7 @@ bool NodeTool::root_handler(GdkEvent* event) {
                 }
             }
         }
+        desktop->canvas->forceFullRedrawAfterInterruptions(5);
         break;
 
     default:
@@ -777,8 +779,8 @@ void NodeTool::select_point(Geom::Point const &/*sel*/, GdkEventButton *event) {
         } else {
             selection->set(item_clicked);
         }
-
-        this->desktop->updateNow();
+        // This not need to be called canvas is updated on selection change
+        // this->desktop->updateNow();
     }
 }