Merge pull request #16175 from jklymak/fix-mouse-ignore-non-visible-axes

jklymak · web-flow · commit b6d333642e2c · 2020-01-10T09:16:15.000-08:00
FIX: ignore axes that aren't visible
diff --git a/lib/matplotlib/_constrained_layout.py b/lib/matplotlib/_constrained_layout.py
@@ -254,10 +254,7 @@ def _make_ghost_gridspec_slots(fig, gs):
             # this gridspec slot doesn't have an axis so we
             # make a "ghost".
             ax = fig.add_subplot(gs[nn])
-            ax.set_frame_on(False)
-            ax.set_xticks([])
-            ax.set_yticks([])
-            ax.set_facecolor((1, 0, 0, 0))
+            ax.set_visible(False)
 
 
 def _make_layout_margins(ax, renderer, h_pad, w_pad):
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
@@ -1832,7 +1832,7 @@ def enter_notify_event(self, guiEvent=None, xy=None):
 
     def inaxes(self, xy):
         """
-        Return the topmost `~.axes.Axes` containing the point *xy*.
+        Return the topmost visible `~.axes.Axes` containing the point *xy*.
 
         Parameters
         ----------
@@ -1842,11 +1842,10 @@ def inaxes(self, xy):
         Returns
         -------
         axes : `~matplotlib.axes.Axes` or None
-            The topmost axes containing the point, or None if no axes.
+            The topmost visible axes containing the point, or None if no axes.
         """
         axes_list = [a for a in self.figure.get_axes()
-                     if a.patch.contains_point(xy)]
-
+                     if a.patch.contains_point(xy) and a.get_visible()]
         if axes_list:
             axes = cbook._topmost_artist(axes_list)
         else:
diff --git a/lib/matplotlib/tests/test_axes.py b/lib/matplotlib/tests/test_axes.py
@@ -6898,3 +6898,11 @@ def test_pi_get_negative_values():
     fig, ax = plt.subplots()
     with pytest.raises(ValueError):
         ax.pie([5, 5, -3], explode=[0, .1, .2])
+
+
+def test_invisible_axes():
+    # invisible axes should not respond to events...
+    fig, ax = plt.subplots()
+    assert fig.canvas.inaxes((200, 200)) is not None
+    ax.set_visible(False)
+    assert fig.canvas.inaxes((200, 200)) is None
