Apply suggestions from code review

dstansby · QuLogic · web-flow · commit 832d55e88d13 · 2021-04-07T10:30:12.000+01:00
Co-authored-by: Elliott Sales de Andrade &lt;quantum.analyst@gmail.com&gt;
diff --git a/doc/users/next_whats_new/widget_dragging.rst b/doc/users/next_whats_new/widget_dragging.rst
@@ -3,7 +3,7 @@ Dragging selectors
 
 The `~matplotlib.widgets.RectangleSelector` and
 `~matplotlib.widgets.EllipseSelector` have a new keyword argument,
-``drag_from_anywhere``, which when set to `True` allows you to click and drag
+*drag_from_anywhere*, which when set to `True` allows you to click and drag
 from anywhere inside the selector to move it. Previously it was only possible
 to move it by either activating the move modifier button, or clicking on the
 central handle.
diff --git a/lib/matplotlib/tests/test_widgets.py b/lib/matplotlib/tests/test_widgets.py
@@ -61,7 +61,7 @@ def onselect(epress, erelease):
     do_event(tool, 'release', xdata=100, ydata=100, button=1)
     assert tool.center == (50, 50)
     # Drag inside rectangle, but away from centre handle to make sure rectangle
-    # is still moved
+    # is still moved, if *drag_from_anywhere* is enabled.
     do_event(tool, 'press', xdata=25, ydata=25, button=1)
     do_event(tool, 'onmove', xdata=75, ydata=75, button=1)
     do_event(tool, 'release', xdata=75, ydata=75, button=1)
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -2551,7 +2551,7 @@ def _set_active_handle(self, event):
         elif m_dist < self.maxdist * 2:
             # Prioritise center handle over other handles
             self.active_handle = 'C'
-        elif (c_dist > self.maxdist and e_dist > self.maxdist):
+        elif c_dist > self.maxdist and e_dist > self.maxdist:
             # Not close to any handles
             if self.drag_from_anywhere and contains:
                 # Check if we've clicked inside the region
