DOC: Whats new entry for title moving above xaxis

jklymak · jklymak · commit c2db893dc6d3 · 2018-04-03T10:06:07.000-07:00
diff --git a/doc/users/next_whats_new/title_will_not_overlap_xaxis.rst b/doc/users/next_whats_new/title_will_not_overlap_xaxis.rst
@@ -1,8 +1,15 @@
 Axes title will no longer overlap xaxis
 ---------------------------------------
 
-Previously the axes title had to be moved manually if an xaxis overlapped
-(usually when the xaxis was put on the top of the axes).  The title can
-still be placed manually.  However, titles that need to be moved are
-at ``y=1.0``,  so manualy placing at 1.0 will be moved, so if the title
-is to be placed at 1.0, it should be set to something near 1.0, like 1.001.
+Previously an axes title had to be moved manually if an xaxis overlapped
+(usually when the xaxis was put on the top of the axes).  Now, the title
+will be automatically moved above the xaxis and its decorators (including
+the xlabel) if they are at the top.
+
+If desired, the title can still be placed manually.  There is a slight kludge;
+the algorithm checks if the y-position of the title is 1.0 (the default),
+and moves if it is.  If the user places the title in the default location
+(i.e. ``ax.title.set_position(0.5, 1.0)``), the title will still be moved
+above the xaxis.  If the user wants to avoid this, they can
+specify a number that is close (i.e. ``ax.title.set_position(0.5, 1.01)``)
+and the title will not be moved via this algorithm.
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -2485,7 +2485,7 @@ def _update_title_position(self, renderer):
                     if (ax.xaxis.get_label_position() == 'top'
                             or ax.xaxis.get_ticks_position() == 'top'):
                         bb = ax.xaxis.get_tightbbox(renderer)
-                        top = bb.y1
+                        top = bb.ymax
                         # we don't need to pad because the padding is already
                         # in __init__: titleOffsetTrans
                         yn = self.transAxes.inverted().transform((0., top))[1]
