Update range selection items to display handles only when resizable

Author: PierreRaybaut

CHANGELOG.md

@@ -41,6 +41,9 @@
   * Setting item movable state now also sets the resizable state:
     * The PlotPy event system won't prevent the user from moving the item by dragging the handles if the item is just not movable: it has to be not resizable, which is not intuitive.
     * This is now fixed
+* Range selection items (`XRangeSelection`, `YRangeSelection`):
+  * Handles are now displayed only when the item is resizable
+  * If the item is set as not resizable (using the `set_resizable` method), the handles will be hidden
 
 Other changes:
 

plotpy/items/shape/range.py

@@ -318,10 +318,11 @@ def draw(
         cx = rct.center().x()
         painter.drawLine(QC.QPointF(cx, rct.top()), QC.QPointF(cx, rct.bottom()))
 
-        painter.setPen(pen)
-        x0, x1, y = self.get_handles_pos()
-        sym.drawSymbol(painter, QC.QPointF(x0, y))
-        sym.drawSymbol(painter, QC.QPointF(x1, y))
+        if self.can_resize():
+            painter.setPen(pen)
+            x0, x1, y = self.get_handles_pos()
+            sym.drawSymbol(painter, QC.QPointF(x0, y))
+            sym.drawSymbol(painter, QC.QPointF(x1, y))
 
     def hit_test(self, pos: QPointF) -> tuple[float, float, bool, None]:
         """Return a tuple (distance, attach point, inside, other_object)
@@ -479,10 +480,11 @@ def draw(
         cy = rct.center().y()
         painter.drawLine(QC.QPointF(rct.left(), cy), QC.QPointF(rct.right(), cy))
 
-        painter.setPen(pen)
-        y0, y1, x = self.get_handles_pos()
-        sym.drawSymbol(painter, QC.QPointF(x, y0))
-        sym.drawSymbol(painter, QC.QPointF(x, y1))
+        if self.can_resize():
+            painter.setPen(pen)
+            y0, y1, x = self.get_handles_pos()
+            sym.drawSymbol(painter, QC.QPointF(x, y0))
+            sym.drawSymbol(painter, QC.QPointF(x, y1))
 
     def hit_test(self, pos: QPointF) -> tuple[float, float, bool, None]:
         """Return a tuple (distance, attach point, inside, other_object)