From e10b9e3c1a8a038c4aed05e37f8909bfa5ff4111 Mon Sep 17 00:00:00 2001
From: "massimo.ferraro" <massimo.ferraro@soft.it>
Date: Wed, 23 Oct 2024 18:50:11 +0200
Subject: [PATCH] Add fix on mouse button and text node selection

Signed-off-by: massimo.ferraro <massimo.ferraro@soft.it>
---
 .../network-area-diagram-viewer.ts             | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/src/components/network-area-diagram-viewer/network-area-diagram-viewer.ts b/src/components/network-area-diagram-viewer/network-area-diagram-viewer.ts
index 792dbb36..d85b1398 100644
--- a/src/components/network-area-diagram-viewer/network-area-diagram-viewer.ts
+++ b/src/components/network-area-diagram-viewer/network-area-diagram-viewer.ts
@@ -325,17 +325,25 @@ export class NetworkAreaDiagramViewer {
     }
 
     private handleStartDragSelectEvent(event: Event) {
+        // check mouse button
+        if ((event as MouseEvent).button !== 0) {
+            return;
+        }
         // check element is draggable or selectable
         const draggableElem = DiagramUtils.getDraggableFrom(event.target as SVGElement);
         if (!draggableElem) {
             return;
         }
+        this.shiftKeyOnMouseDown = !!(event as MouseEvent).shiftKey;
+        // avoid selecting text nodes
+        if (this.shiftKeyOnMouseDown && DiagramUtils.isTextNode(draggableElem as SVGGraphicsElement)) {
+            return;
+        }
         this.disablePanzoom(); // to avoid panning the whole SVG when moving or selecting a node
         // change cursor style
         const svg: HTMLElement = <HTMLElement>this.svgDraw?.node.firstElementChild?.parentElement;
         svg.style.cursor = 'grabbing';
         // check dragging vs. selection
-        this.shiftKeyOnMouseDown = !!(event as MouseEvent).shiftKey;
         if (!this.shiftKeyOnMouseDown) {
             // moving node
             this.initializeDrag(draggableElem);
@@ -357,6 +365,10 @@ export class NetworkAreaDiagramViewer {
     }
 
     private handleDragEvent(event: Event) {
+        // check mouse button
+        if ((event as MouseEvent).button !== 0) {
+            return;
+        }
         if (this.selectedElement && !this.shiftKeyOnMouseDown) {
             event.preventDefault();
             this.ctm = this.svgDraw?.node.getScreenCTM(); // used to compute SVG transformations
@@ -403,6 +415,10 @@ export class NetworkAreaDiagramViewer {
     }
 
     private handleEndDragSelectEvent(event: Event) {
+        // check mouse button
+        if ((event as MouseEvent).button !== 0) {
+            return;
+        }
         // check if I moved or selected an element
         if (this.selectedElement) {
             if (!this.shiftKeyOnMouseDown) {