Skip to content

Add an rc file setting for the pad parameter of tight_layout #5266

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
itoijala opened this issue Oct 17, 2015 · 6 comments
Closed

Add an rc file setting for the pad parameter of tight_layout #5266

itoijala opened this issue Oct 17, 2015 · 6 comments
Assignees
@tacaswell
Labels
New feature topic: rcparams

Comments

@itoijala
Copy link
Contributor

Currently the parameter defaults to pad=1.08 with no way to change it from the rc file, for example when using figure.autolayout, which is then not so useful since the user has to call tight_layout anyway to set the padding.
@itoijala itoijala mentioned this issue Oct 18, 2015
Closed
@tacaswell tacaswell self-assigned this Jan 25, 2016
@jklymak jklymak added Good first issue Open a pull request against these issues if there are no active ones! topic: rcparams New feature and removed status: needs review labels Aug 17, 2018
@ConnorBrooks8 ConnorBrooks8 mentioned this issue Oct 2, 2018
Closed
6 tasks
@Uttam-Singhh
Copy link

Hello, Can I work on this?

@jklymak
Copy link
Member

jklymak commented May 28, 2021

I actually don't think that would be a good idea. Right now we are planning to redo how layout engines work, and adding new state to the rcParams file would make that harder in my opinion. See #19892

@jklymak jklymak removed the Good first issue Open a pull request against these issues if there are no active ones! label May 28, 2021
@jklymak
Copy link
Member

jklymak commented May 28, 2021

BTW< thanks for being willing to help! Perhaps there are other issues. Feel free to drop in at https://gitter.im/matplotlib/matplotlib if you'd like to chat...

@Uttam-Singhh
Copy link

Oh ok! no problem

@github-actions
Copy link

This issue has been marked "inactive" because it has been 365 days since the last comment. If this issue is still present in recent Matplotlib releases, or the feature request is still wanted, please leave a comment and this label will be removed. If there are no updates in another 30 days, this issue will be automatically closed, but you are free to re-open or create a new issue if needed. We value issue reports, and this procedure is meant to help us resurface and prioritize issues that have not been addressed yet, not make them disappear. Thanks for your help!

@github-actions github-actions bot added the status: inactive Marked by the “Stale” Github Action label Mar 17, 2023
@rcomer
Copy link
Member

rcomer commented Mar 17, 2023

I’m guessing this is now redundant as we have constrained layout, which does take its default padding from rcParams. I also note that, as of #25395, use of tight layout is “mildly discouraged“.

@rcomer rcomer closed this as not planned Won't fix, can't repro, duplicate, stale Mar 17, 2023
@rcomer rcomer removed the status: inactive Marked by the “Stale” Github Action label Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tacaswell tacaswell

Labels
New feature topic: rcparams
Projects
None yet
Milestone
No milestone
5 participants
@tacaswell @itoijala @jklymak @rcomer @Uttam-Singhh