updated the autotick label faq to use bboxes and transforms rather than iteration

jdh2358 · jdh2358 · commit 5e8b1be94032 · 2008-06-25T21:43:09.000Z
svn path=/trunk/matplotlib/; revision=5673
diff --git a/doc/faq/howto_faq.rst b/doc/faq/howto_faq.rst
@@ -112,9 +112,9 @@ connecting
 get the window extent there, and then do something with it, eg move
 the left of the canvas over; see :ref:`event-handling-tutorial`.
 
-Here is a recursive, iterative solution that will gradually move the
-left of the subplot over until the label fits w/o going outside the
-figure border (requires matplotlib 0.98)
+Here is that gets a bounding box in relative figure coordinates (0..1)
+of each of the labels and uses it to move the left of the subplots
+over so that the tick labels fit in the figure
 
 .. plot:: auto_subplots_adjust.py
    :include-source:
diff --git a/doc/pyplots/auto_subplots_adjust.py b/doc/pyplots/auto_subplots_adjust.py
@@ -1,19 +1,30 @@
 import matplotlib.pyplot as plt
-
+import matplotlib.transforms as mtransforms
 fig = plt.figure()
 ax = fig.add_subplot(111)
 ax.plot(range(10))
 ax.set_yticks((2,5,7))
 labels = ax.set_yticklabels(('really, really, really', 'long', 'labels'))
 
 def on_draw(event):
+   bboxes = []
    for label in labels:
        bbox = label.get_window_extent()
-       if bbox.xmin<0:
-           fig.subplots_adjust(left=1.1*fig.subplotpars.left)
-           fig.canvas.draw()
-           break
+       # the figure transform goes from relative coords->pixels and we
+       # want the inverse of that
+       bboxi = bbox.inverse_transformed(fig.transFigure)
+       bboxes.append(bboxi)
+
+   # this is the bbox that bounds all the bboxes, again in relative
+   # figure coords
+   bbox = mtransforms.Bbox.union(bboxes)
+   if fig.subplotpars.left < bbox.width:
+       # we need to move it over
+       fig.subplots_adjust(left=1.1*bbox.width) # pad a little
+       fig.canvas.draw()
+   return False
 
 fig.canvas.mpl_connect('draw_event', on_draw)
 
 plt.show()
+
