Switch the cursor to a busy cursor while redrawing.

anntzer · anntzer · commit 004bf6061201 · 2017-05-31T15:23:17.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
@@ -30,7 +30,7 @@
 from math import radians, cos, sin
 from matplotlib import verbose, rcParams, __version__
 from matplotlib.backend_bases import (RendererBase, FigureManagerBase,
-                                      FigureCanvasBase)
+                                      FigureCanvasBase, cursors)
 from matplotlib.cbook import maxdict, restrict_dict
 from matplotlib.figure import Figure
 from matplotlib.font_manager import findfont, get_font
@@ -471,9 +471,14 @@ def draw(self):
         # acquire a lock on the shared font cache
         RendererAgg.lock.acquire()
 
+        toolbar = self.toolbar
         try:
+            if toolbar:
+                toolbar.set_cursor(cursors.WAIT)
             self.figure.draw(self.renderer)
         finally:
+            if toolbar:
+                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
@@ -59,6 +59,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
@@ -100,6 +100,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
@@ -46,6 +46,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
@@ -1548,6 +1548,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
`@@ -59,6 +59,7 @@ def fn_name(): return sys._getframe(1).f_code.co_name`
`59`	`59`	`cursors.HAND : gdk.Cursor(gdk.HAND2),`
`60`	`60`	`cursors.POINTER : gdk.Cursor(gdk.LEFT_PTR),`
`61`	`61`	`cursors.SELECT_REGION : gdk.Cursor(gdk.TCROSS),`
	`62`	`+ cursors.WAIT : gdk.Cursor(gdk.WATCH),`
`62`	`63`	`}`
`63`	`64`
`64`	`65`	`# 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
`@@ -100,6 +100,7 @@ def fn_name():`
`100`	`100`	`cursors.HAND: QtCore.Qt.PointingHandCursor,`
`101`	`101`	`cursors.POINTER: QtCore.Qt.ArrowCursor,`
`102`	`102`	`cursors.SELECT_REGION: QtCore.Qt.CrossCursor,`
	`103`	`+ cursors.WAIT: QtCore.Qt.WaitCursor,`
`103`	`104`	`}`
`104`	`105`
`105`	`106`
Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,7 @@`
`46`	`46`	`cursors.HAND: "hand2",`
`47`	`47`	`cursors.POINTER: "arrow",`
`48`	`48`	`cursors.SELECT_REGION: "tcross",`
	`49`	`+ cursors.WAIT: "watch",`
`49`	`50`	`}`
`50`	`51`
`51`	`52`
Original file line number	Diff line number	Diff line change
`@@ -1548,6 +1548,7 @@ def updateButtonText(self, lst):`
`1548`	`1548`	`cursors.HAND: wx.CURSOR_HAND,`
`1549`	`1549`	`cursors.POINTER: wx.CURSOR_ARROW,`
`1550`	`1550`	`cursors.SELECT_REGION: wx.CURSOR_CROSS,`
	`1551`	`+ cursors.WAIT: wx.CURSOR_WAIT,`
`1551`	`1552`	`}`
`1552`	`1553`
`1553`	`1554`