Switch the cursor to a busy cursor while redrawing.

anntzer · anntzer · commit a9376df6c53e · 2017-08-23T09:27:33.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
@@ -24,7 +24,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 (
-    _Backend, FigureCanvasBase, FigureManagerBase, RendererBase)
+    _Backend, FigureCanvasBase, FigureManagerBase, RendererBase, cursors)
 from matplotlib.cbook import maxdict
 from matplotlib.figure import Figure
 from matplotlib.font_manager import findfont, get_font
@@ -422,9 +422,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
@@ -54,6 +54,7 @@
     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
@@ -51,6 +51,7 @@
     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),
     }
 
 
diff --git a/lib/matplotlib/backends/backend_qt5.py b/lib/matplotlib/backends/backend_qt5.py
@@ -90,6 +90,7 @@
     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
@@ -45,6 +45,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
@@ -1474,6 +1474,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
`@@ -54,6 +54,7 @@`
`54`	`54`	`cursors.HAND : gdk.Cursor(gdk.HAND2),`
`55`	`55`	`cursors.POINTER : gdk.Cursor(gdk.LEFT_PTR),`
`56`	`56`	`cursors.SELECT_REGION : gdk.Cursor(gdk.TCROSS),`
	`57`	`+ cursors.WAIT : gdk.Cursor(gdk.WATCH),`
`57`	`58`	`}`
`58`	`59`
`59`	`60`	`# ref gtk+/gtk/gtkwidget.h`
Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@`
`51`	`51`	`cursors.HAND : Gdk.Cursor.new(Gdk.CursorType.HAND2),`
`52`	`52`	`cursors.POINTER : Gdk.Cursor.new(Gdk.CursorType.LEFT_PTR),`
`53`	`53`	`cursors.SELECT_REGION : Gdk.Cursor.new(Gdk.CursorType.TCROSS),`
	`54`	`+ cursors.WAIT : Gdk.Cursor.new(Gdk.CursorType.WATCH),`
`54`	`55`	`}`
`55`	`56`
`56`	`57`
Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,7 @@`
`90`	`90`	`cursors.HAND: QtCore.Qt.PointingHandCursor,`
`91`	`91`	`cursors.POINTER: QtCore.Qt.ArrowCursor,`
`92`	`92`	`cursors.SELECT_REGION: QtCore.Qt.CrossCursor,`
	`93`	`+ cursors.WAIT: QtCore.Qt.WaitCursor,`
`93`	`94`	`}`
`94`	`95`
`95`	`96`
Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,7 @@`
`45`	`45`	`cursors.HAND: "hand2",`
`46`	`46`	`cursors.POINTER: "arrow",`
`47`	`47`	`cursors.SELECT_REGION: "tcross",`
	`48`	`+ cursors.WAIT: "watch",`
`48`	`49`	`}`
`49`	`50`
`50`	`51`
Original file line number	Diff line number	Diff line change
`@@ -1474,6 +1474,7 @@ def updateButtonText(self, lst):`
`1474`	`1474`	`cursors.HAND: wx.CURSOR_HAND,`
`1475`	`1475`	`cursors.POINTER: wx.CURSOR_ARROW,`
`1476`	`1476`	`cursors.SELECT_REGION: wx.CURSOR_CROSS,`
	`1477`	`+ cursors.WAIT: wx.CURSOR_WAIT,`
`1477`	`1478`	`}`
`1478`	`1479`
`1479`	`1480`