Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Improve layout of Fill Betweenx Demo#12000
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Improve layout of Fill Betweenx Demo#12000
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| @@ -13,35 +14,35 @@ | |||
| x1 = np.sin(2 * np.pi * y) | |||
| x2 = 1.2 * np.sin(4 * np.pi * y) | |||
| fig, [ax1, ax2, ax3] = plt.subplots(3, 1, sharex=True) | |||
| fig, [ax1, ax2, ax3] = plt.subplots(1, 3, sharey=True, figsize=(6, 6)) | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Would this demo be helped by constrained_layout=True?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Not in particular. The main issue is that the suplots were stacked vertically. This squeezes the y axis stongly so that it's hard to see what's going on. I just changed to stacking horizontally. This looks ok even without constrainedlayout. I've linked the result in the first post above.
…000-on-v2.2.3-docBackport PR#12000 on branch v2.2.3-doc (Improve layout of Fill Betweenx Demo)
Uh oh!
There was an error while loading.Please reload this page.
PR Summary
The layout of theFill Betweenx Demo is a bit cramped.
This PR fixes this by changing from a row layout to a column layout. It includes some minor text cleanup as well.
New layout