From 9de62ded3d16d35ab3b9776d7e7f4bfb681d3886 Mon Sep 17 00:00:00 2001
From: "Christopher L. Farrow" <cfarrow@enthought.com>
Date: Mon, 29 Sep 2014 15:11:56 -0500
Subject: [PATCH 1/2] FIX: Properly clip bounds when leaving selection area.

---
 chaco/tools/range_selection.py                | 11 +++-
 chaco/tools/tests/__init__.py                 |  0
 .../tools/tests/range_selection_test_case.py  | 57 +++++++++++++++++++
 3 files changed, 66 insertions(+), 2 deletions(-)
 create mode 100644 chaco/tools/tests/__init__.py
 create mode 100644 chaco/tools/tests/range_selection_test_case.py

diff --git a/chaco/tools/range_selection.py b/chaco/tools/range_selection.py
index 538423730..3ad1e8a72 100644
--- a/chaco/tools/range_selection.py
+++ b/chaco/tools/range_selection.py
@@ -510,9 +510,16 @@ def selecting_mouse_leave(self, event):
 
         pos = self._get_axis_coord(event)
         if pos >= high:
-            selection_high = self.mapper.map_data(high)
+            # clip to the boundary appropriate for the mapper's orientation.
+            if self.mapper.sign == 1:
+                selection_high = self.mapper.map_data(high)
+            else:
+                selection_high = self.mapper.map_data(low)
         elif pos <= low:
-            selection_low = self.mapper.map_data(low)
+            if self.mapper.sign == 1:
+                selection_low = self.mapper.map_data(low)
+            else:
+                selection_low = self.mapper.map_data(high)
 
         self.selection = (selection_low, selection_high)
         event.window.set_pointer("arrow")
diff --git a/chaco/tools/tests/__init__.py b/chaco/tools/tests/__init__.py
new file mode 100644
index 000000000..e69de29bb
diff --git a/chaco/tools/tests/range_selection_test_case.py b/chaco/tools/tests/range_selection_test_case.py
new file mode 100644
index 000000000..c94fb54af
--- /dev/null
+++ b/chaco/tools/tests/range_selection_test_case.py
@@ -0,0 +1,57 @@
+import unittest
+
+import numpy as np
+
+from chaco.array_plot_data import ArrayPlotData
+from chaco.plot import Plot
+from chaco.tools.range_selection import RangeSelection
+from enable.testing import EnableTestAssistant
+
+
+class RangeSelectionTestCase(EnableTestAssistant, unittest.TestCase):
+
+    def test_selecting_mouse_leave_clipping(self):
+        # Regression test for #216.
+        plot_data = ArrayPlotData()
+        arr = np.arange(4.0)
+        plot_data.set_data("x", arr)
+        plot_data.set_data("y", arr)
+
+        for origin in ('bottom left', 'top left', 'bottom right', 'top right'):
+            for orientation in ('h', 'v'):
+                for axis in ('index', 'value'):
+                    plot = Plot(
+                        plot_data, orientation=orientation, origin='top right'
+                    )
+
+                    renderer = plot.plot(('x', 'y'))[0]
+                    renderer.bounds = [10, 20]
+                    tool = RangeSelection(
+                        renderer, left_button_selects=True, axis=axis,
+                    )
+                    renderer.tools.append(tool)
+
+                    low_x, low_y = plot.position
+                    high_x = low_x + renderer.bounds[0] - 1
+                    high_y = low_y + renderer.bounds[1] - 1
+
+                    cx = 5
+                    cy = 5
+
+                    bounds = (
+                        (low_x - 1, low_y),
+                        (high_x + 1, low_y),
+                        (low_x, low_y - 1),
+                        (low_x, high_y + 1),
+                    )
+                    for x, y in bounds:
+                        self.mouse_down(tool, x=cx, y=cy)
+                        self.mouse_leave(tool, x=x, y=y)
+                        selection = tool.selection
+                        self.assertTrue(selection[0] <= selection[1])
+                        self.mouse_up(tool, x=x, y=y)
+
+
+if __name__ == '__main__':
+    import nose
+    nose.run()

From 9918f22692f4a61bf44869d1bd3e18b8630d2ceb Mon Sep 17 00:00:00 2001
From: "Christopher L. Farrow" <cfarrow@enthought.com>
Date: Mon, 29 Sep 2014 15:12:58 -0500
Subject: [PATCH 2/2] Updating CHANGES.txt.

---
 CHANGES.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGES.txt b/CHANGES.txt
index f0d6e9399..2c8264274 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -23,6 +23,7 @@ Fixes
  * Resume running test for empty container that has been skipped (PR#190).
  * Improved handling of missing or empty data in scatterplot (PR#210)
  * Avoid a numpy crash when using a data source of a unicode array (PR#213).
+ * RangeSelection properly clips bounds when leaving selection area (PR#217).
 
 Release 4.4.1
 -------------