#223 Fix horizontal / vertical edge selection.

Signed-off-by: cneben <[email protected]>
cneben · Feb 20, 2024 · f6f6b6a · f6f6b6a
1 parent ae6ea81
commit f6f6b6a
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/src/qanGraphView.cpp b/src/qanGraphView.cpp
@@ -33,6 +33,7 @@
 //-----------------------------------------------------------------------------
 
 // Qt headers
+#include <QtNumeric>
 #include <QQuickItem>
 
 // QuickQanava headers
@@ -146,8 +147,12 @@ void    GraphView::selectionRectActivated(const QRectF& rect)
             auto edgeItem = qobject_cast<qan::EdgeItem*>(item);
             if (edgeItem != nullptr &&
                 edgeItem->getEdge() != nullptr) {
-                const auto itemBr = item->mapRectToItem(_graph->getContainerItem(),
+                auto itemBr = item->mapRectToItem(_graph->getContainerItem(),
                                                         item->boundingRect());
+                if (qFuzzyIsNull(itemBr.height()))  // Note: rect.contains does not work with 0 width/height
+                    itemBr.setHeight(0.5);          // br, set minimum width/height to 0.5, to allow vertical /
+                if (qFuzzyIsNull(itemBr.width()))   // horizontal line selection for example.
+                    itemBr.setWidth(0.5);
                 if (!rect.contains(itemBr)) {
                     _graph->setEdgeSelected(edgeItem->getEdge(), false);
                     _selectedItems.remove(item);
@@ -177,11 +182,14 @@ void    GraphView::selectionRectActivated(const QRectF& rect)
             auto edgeItem = qobject_cast<qan::EdgeItem*>(item);
             if (edgeItem != nullptr &&
                 edgeItem->getEdge() != nullptr) {
-                const auto itemBr = item->mapRectToItem(_graph->getContainerItem(),
-                                                        item->boundingRect());
+                auto itemBr = item->mapRectToItem(_graph->getContainerItem(),
+                                                  item->boundingRect());
+                if (qFuzzyIsNull(itemBr.height()))  // Note: rect.contains does not work with 0 width/height
+                    itemBr.setHeight(0.5);          // br, set minimum width/height to 0.5, to allow vertical /
+                if (qFuzzyIsNull(itemBr.width()))   // horizontal line selection for example.
+                    itemBr.setWidth(0.5);
                 if (rect.contains(itemBr)) {
                     auto edge = edgeItem->getEdge();
-
                     _graph->setEdgeSelected(edge, true);
                     _selectedItems.insert(edgeItem);
                 }