Skip to content

Add legend.titlesize to example rc file #28

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

kilojoules
Copy link

No description provided.

@ImportanceOfBeingErnest
Copy link
Member

Note that this is the documentation repo. It is more or less just the host for the webpage. Changes need to go into the source repo at github.com/matplotlib/matplotlib.

Second, note that legend.titlesize is not a valid rc parameter. You probably mean legend.title_fontsize. This parameter already is part of the rc file

https://github.com/matplotlib/matplotlib/blob/dbaf19d6410d3c92656787405ee4238b271c53c2/matplotlibrc.template#L423

So the _static/matplotlibrc file here is hence an old version of the real rc file that is continuously updated.

The real question would be: Can we safely delete this file or is it accessed from anywhere?

@QuLogic
Copy link
Member

QuLogic commented Mar 5, 2019

It was used with literalinclude to fill out https://matplotlib.org/tutorials/introductory/customizing.html, but it's since been moved to point to matplotlibrc.template. The only reference to this file is now in documentation about writing documentation. We should probably delete, but that's basically part the larger project of deleting/redirecting old files.

As this repo is not the source for this file, I am going to close this PR.

@QuLogic QuLogic closed this Mar 5, 2019
@kilojoules
Copy link
Author

@ImportanceOfBeingErnest Just FYI the first thing that comes up from a search is https://matplotlib.org/users/customizing.html#matplotlib-rcparams, which does not include the title_fontsize option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants