FIX: move titles up if overlap axes

jklymak · jklymak · commit 7935c5a6225a · 2018-07-03T13:25:26.000-07:00
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -2478,27 +2478,38 @@ def _update_title_position(self, renderer):
         ymax = -10
         for title in titles:
             x, y0 = title.get_position()
-            y = y0
-            bt = title.get_window_extent(renderer)
+            y = 1
+            # need to start agin in case of window resizing
+            title.set_position((x, 1.0))
             # need to check all our twins too...
             axs = self._twinned_axes.get_siblings(self)
 
+            top = 0
             for ax in axs:
                 try:
-                    bb = ax.get_window_extent(renderer)
                     if (ax.xaxis.get_label_position() == 'top'
                             or ax.xaxis.get_ticks_position() == 'top'):
-                        bb = mtransforms.Bbox.union([bb,
-                                ax.xaxis.get_tightbbox(renderer)])
-                    # need to do twice to get right for manual resizing
-                    dy = bt.ymin - bb.ymax
-                    if dy < 0:
-                        y = self.transAxes.inverted().transform(
-                                (0., bb.ymax - dy))[1]
-                        title.set_position((x, y))
-                        #bt = title.get_window_extent(renderer)
+                        bb = ax.xaxis.get_tightbbox(renderer)
+                    else:
+                        bb = ax.get_window_extent(renderer)
+                    top = np.max((top, bb.ymax))
                 except AttributeError:
+                    # this happens for an empty bb
                     y = 1
+            bt = title.get_window_extent(renderer)
+            # need to do twice to get right for manual resizing
+            for nn in range(1):
+                dy = bt.ymin - top
+                if dy < 0:
+                    y = self.transAxes.inverted().transform(
+                            (0., top))[1]
+                    title.set_position((x, y))
+                    bt = title.get_window_extent(renderer)
+                    dy = bt.ymin - top
+                    if dy < 0:
+                        y = self.transAxes.inverted().transform(
+                            (0., top - dy))[1]
+                    title.set_position((x, y))
             ymax = np.max((y, ymax))
         for title in titles:
             # now line up all the titles at the highest baseline.
