From 5a2dae366dd8d2797b55b0cd0eb801d2e03f0176 Mon Sep 17 00:00:00 2001
From: Jabier Arraiza <jabier.arraiza@marker.es>
Date: Wed, 9 Oct 2019 06:05:56 +0000
Subject: Improve node selection ~x3 removing unneded updateNow() called in non
 necesary places. Also ensure canvas is always redraw

---
 src/ui/tools/node-tool.cpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'src/ui/tools/node-tool.cpp')

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();
     }
 }
 
-- 
cgit v1.2.3