Switch the cursor to a busy cursor while redrawing.

anntzer · anntzer · commit 6c8450c88372 · 2016-06-18T00:49:38.000-07:00
To test, plot e.g. `plot(rand(100000))` and zoom in manually (the
redraw should take some time).  The cursor should switch to a "busy"
cursor (e.g. spinwheel).  The switch doesn't seem to happen under the
Tk and WX backends, probably due to some event loop intricacies I don't
understand.
diff --git a/lib/matplotlib/backend_tools.py b/lib/matplotlib/backend_tools.py
@@ -23,7 +23,7 @@
 
 class Cursors(object):
     """Simple namespace for cursor reference"""
-    HAND, POINTER, SELECT_REGION, MOVE = list(range(4))
+    HAND, POINTER, SELECT_REGION, MOVE, WAIT = list(range(5))
 cursors = Cursors()
 
 # Views positions tool
diff --git a/lib/matplotlib/backends/backend_agg.py b/lib/matplotlib/backends/backend_agg.py
@@ -29,7 +29,7 @@
 from math import radians, cos, sin
 from matplotlib import verbose, rcParams
 from matplotlib.backend_bases import (RendererBase, FigureManagerBase,
-                                      FigureCanvasBase)
+                                      FigureCanvasBase, cursors)
 from matplotlib.cbook import is_string_like, maxdict, restrict_dict
 from matplotlib.figure import Figure
 from matplotlib.font_manager import findfont, get_font
@@ -461,8 +461,10 @@ def draw(self):
         RendererAgg.lock.acquire()
 
         try:
+            self.toolbar.set_cursor(cursors.WAIT)
             self.figure.draw(self.renderer)
         finally:
+            self.toolbar.set_cursor(cursors.POINTER)
             RendererAgg.lock.release()
 
     def get_renderer(self, cleared=False):
diff --git a/lib/matplotlib/backends/backend_gtk.py b/lib/matplotlib/backends/backend_gtk.py
@@ -58,6 +58,7 @@ def fn_name(): return sys._getframe(1).f_code.co_name
     cursors.HAND          : gdk.Cursor(gdk.HAND2),
     cursors.POINTER       : gdk.Cursor(gdk.LEFT_PTR),
     cursors.SELECT_REGION : gdk.Cursor(gdk.TCROSS),
+    cursors.WAIT          : gdk.Cursor(gdk.WATCH),
     }
 
 # ref gtk+/gtk/gtkwidget.h
diff --git a/lib/matplotlib/backends/backend_gtk3.py b/lib/matplotlib/backends/backend_gtk3.py
@@ -55,6 +55,7 @@ def fn_name(): return sys._getframe(1).f_code.co_name
     cursors.HAND          : Gdk.Cursor.new(Gdk.CursorType.HAND2),
     cursors.POINTER       : Gdk.Cursor.new(Gdk.CursorType.LEFT_PTR),
     cursors.SELECT_REGION : Gdk.Cursor.new(Gdk.CursorType.TCROSS),
+    cursors.WAIT          : Gdk.Cursor.new(Gdk.CursorType.WATCH),
     }
 
 def draw_if_interactive():
diff --git a/lib/matplotlib/backends/backend_qt5.py b/lib/matplotlib/backends/backend_qt5.py
@@ -102,6 +102,7 @@ def fn_name():
     cursors.HAND: QtCore.Qt.PointingHandCursor,
     cursors.POINTER: QtCore.Qt.ArrowCursor,
     cursors.SELECT_REGION: QtCore.Qt.CrossCursor,
+    cursors.WAIT: QtCore.Qt.WaitCursor,
     }
 
 
diff --git a/lib/matplotlib/backends/backend_tkagg.py b/lib/matplotlib/backends/backend_tkagg.py
@@ -47,6 +47,7 @@
     cursors.HAND: "hand2",
     cursors.POINTER: "arrow",
     cursors.SELECT_REGION: "tcross",
+    cursors.WAIT: "watch",
     }
 
 
diff --git a/lib/matplotlib/backends/backend_wx.py b/lib/matplotlib/backends/backend_wx.py
@@ -1543,6 +1543,7 @@ def updateButtonText(self, lst):
     cursors.HAND: wx.CURSOR_HAND,
     cursors.POINTER: wx.CURSOR_ARROW,
     cursors.SELECT_REGION: wx.CURSOR_CROSS,
+    cursors.WAIT: wx.CURSOR_WAIT,
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ def fn_name(): return sys._getframe(1).f_code.co_name`
`58`	`58`	`cursors.HAND : gdk.Cursor(gdk.HAND2),`
`59`	`59`	`cursors.POINTER : gdk.Cursor(gdk.LEFT_PTR),`
`60`	`60`	`cursors.SELECT_REGION : gdk.Cursor(gdk.TCROSS),`
	`61`	`+ cursors.WAIT : gdk.Cursor(gdk.WATCH),`
`61`	`62`	`}`
`62`	`63`
`63`	`64`	`# ref gtk+/gtk/gtkwidget.h`
Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@ def fn_name(): return sys._getframe(1).f_code.co_name`
`55`	`55`	`cursors.HAND : Gdk.Cursor.new(Gdk.CursorType.HAND2),`
`56`	`56`	`cursors.POINTER : Gdk.Cursor.new(Gdk.CursorType.LEFT_PTR),`
`57`	`57`	`cursors.SELECT_REGION : Gdk.Cursor.new(Gdk.CursorType.TCROSS),`
	`58`	`+ cursors.WAIT : Gdk.Cursor.new(Gdk.CursorType.WATCH),`
`58`	`59`	`}`
`59`	`60`
`60`	`61`	`def draw_if_interactive():`
Original file line number	Diff line number	Diff line change
`@@ -102,6 +102,7 @@ def fn_name():`
`102`	`102`	`cursors.HAND: QtCore.Qt.PointingHandCursor,`
`103`	`103`	`cursors.POINTER: QtCore.Qt.ArrowCursor,`
`104`	`104`	`cursors.SELECT_REGION: QtCore.Qt.CrossCursor,`
	`105`	`+ cursors.WAIT: QtCore.Qt.WaitCursor,`
`105`	`106`	`}`
`106`	`107`
`107`	`108`
Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@`
`47`	`47`	`cursors.HAND: "hand2",`
`48`	`48`	`cursors.POINTER: "arrow",`
`49`	`49`	`cursors.SELECT_REGION: "tcross",`
	`50`	`+ cursors.WAIT: "watch",`
`50`	`51`	`}`
`51`	`52`
`52`	`53`
Original file line number	Diff line number	Diff line change
`@@ -1543,6 +1543,7 @@ def updateButtonText(self, lst):`
`1543`	`1543`	`cursors.HAND: wx.CURSOR_HAND,`
`1544`	`1544`	`cursors.POINTER: wx.CURSOR_ARROW,`
`1545`	`1545`	`cursors.SELECT_REGION: wx.CURSOR_CROSS,`
	`1546`	`+ cursors.WAIT: wx.CURSOR_WAIT,`
`1546`	`1547`	`}`
`1547`	`1548`
`1548`	`1549`