Compressed layout moves suptitle #28734
Conversation
6cd82cb to
68903da
Compare
|
I belatedly found the plot_children function so can now visualise that I'm using the blue rectangle to position the title.
|
| suptitle.get_in_layout() and suptitle._autopos): | ||
| x, _ = suptitle.get_position() | ||
| suptitle.set_position( | ||
| (x, layoutgrids[fig].get_inner_bbox().y1 + 0.02)) |
There was a problem hiding this comment.
Where does 0.02 come from? Does it work well with different font sizes?
There was a problem hiding this comment.
0.02 is the default space between the top of this title and the top of the figure without constrained layout, so I chose it to be consistent with that.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Though thinking about it, it's probably better to do something consistent with the make_margin_suptitles function which would define the spacing if you are using normal constrained layout or if compressed layout is compressing things horizontally. That ultimately uses the configurable h_pad.
There was a problem hiding this comment.
Here's what happens with my current (0.02) implementation if compressed layout is not moving axes in the vertical and I do not use bbox_inches='tight'. I will change it to use h_pad.
There was a problem hiding this comment.
With the update, here is what I get with figsize=(5, 5) and no bbox_inches='tight'. This looks identical to what I get with main:
Here is what I get with figsize=(4, 10) and saving with bbox_inches='tight', pad_inches='layout':
68903da to
146ba53
Compare
|
Docs are broken on |
PR summary
Closes #27866. I found myself wanting this today. The code from the issue now produces
I am at a loss how to provide a deprecation path for this.
PR checklist