Try to improve the cursor readout and status message.

efiring · efiring · commit bbbb267295aa · 2009-06-06T18:21:51.000Z
This could be done better with changes to the backends.
The status (pan/zoom etc) should be indicated via the
state of the buttons, e.g. using radio buttons, rather
than by printing text.  The readout should be positioned
to minimize jumping around of the text.

svn path=/trunk/matplotlib/; revision=7188
diff --git a/lib/matplotlib/axes.py b/lib/matplotlib/axes.py
@@ -2468,12 +2468,14 @@ def format_ydata(self, y):
     def format_coord(self, x, y):
         'return a format string formatting the *x*, *y* coord'
         if x is None:
-            x = '???'
+            xs = '???'
+        else:
+            xs = self.format_xdata(x)
         if y is None:
-            y = '???'
-        xs = self.format_xdata(x)
-        ys = self.format_ydata(y)
-        return  'x=%s, y=%s'%(xs,ys)
+            ys = '???'
+        else:
+            ys = self.format_ydata(y)
+        return  'x=%s y=%s'%(xs,ys)
 
     #### Interactive manipulation
 
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
@@ -1460,7 +1460,7 @@ def print_figure(self, filename, dpi=None, facecolor='w', edgecolor='w',
 
             restore_bbox = tight_bbox.adjust_bbox(self.figure, format,
                                                   bbox_inches)
-            
+
             _bbox_inches_restore = (bbox_inches, restore_bbox)
         else:
             _bbox_inches_restore = None
@@ -1896,7 +1896,7 @@ def mouse_move(self, event):
             except OverflowError: pass
             else:
                 if len(self.mode):
-                    self.set_message('%s : %s' % (self.mode, s))
+                    self.set_message('%s, %s' % (self.mode, s))
                 else:
                     self.set_message(s)
         else: self.set_message(self.mode)
@@ -1923,7 +1923,7 @@ def pan(self,*args):
                 'button_press_event', self.press_pan)
             self._idRelease = self.canvas.mpl_connect(
                 'button_release_event', self.release_pan)
-            self.mode = 'pan/zoom mode'
+            self.mode = 'pan/zoom'
             self.canvas.widgetlock(self)
         else:
             self.canvas.widgetlock.release(self)
@@ -2193,7 +2193,7 @@ def zoom(self, *args):
         if  self._active:
             self._idPress = self.canvas.mpl_connect('button_press_event', self.press_zoom)
             self._idRelease = self.canvas.mpl_connect('button_release_event', self.release_zoom)
-            self.mode = 'Zoom to rect mode'
+            self.mode = 'zoom rect'
             self.canvas.widgetlock(self)
         else:
             self.canvas.widgetlock.release(self)
diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
@@ -344,7 +344,7 @@ def set_powerlimits(self, lims):
 
     def format_data_short(self,value):
         'return a short formatted string representation of a number'
-        return '%1.3g'%value
+        return '%-12g'%value
 
     def format_data(self,value):
         'return a formatted string representation of a number'
@@ -525,7 +525,7 @@ def format_data(self,value):
 
     def format_data_short(self,value):
         'return a short formatted string representation of a number'
-        return '%1.3g'%value
+        return '%-12g'%value
 
     def is_decade(self, x):
         n = self.nearest_long(x)
