| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Thanks you for this PR! However, as those that you modified here are module names, I am not sure that they should be modified, at least not like this. (Also, I think the style guide indicates that "Title Case" should not be used. https://matplotlib.org/stable/devel/documenting_mpl.html#formatting-and-style-conventions Rather "Specialty Plots" should probably be changed to "Specialty plots".) I do agree with you though that the current approach looks a bit weird. Maybe one should format the modules as "code", so ``pyplot`` ======= etc, but not sure exactly what that would look like. |
Sorry, something went wrong.
|
It is not clear what a "top-level chapter" is in For everything but top-level chapters but looking the current documentation it seems like there are no top-level chapters... |
Sorry, something went wrong.
|
I've now changed it to Upper lower. Regarding module names: I think the style guide saies that "`" should not be used. |
Sorry, something went wrong.
|
Please don't change the capitalization or remove underscores from module names (you can't import Pyplot or import Axes grid). If you want to make these more homogeneous, I would write e.g. The pyplot module/The axes_grid1 module. |
Sorry, something went wrong.
My changes are affecting the example gallery, so I thought that the example gallery captions should have names that describe to the users what examples to expect. Are the example gallery captions the right place for module names? I think module names should only be used in the reference headings:
Yes, but so one can not do import Statistics from the Statistics section |
Sorry, something went wrong.
Indeed, but pyplot is nothing but a module name, whereas Statistics is obviously a common noun. |
Sorry, something went wrong.
|
Just to confirm that what @anntzer maybe expressed clearer, is exactly what I was trying to say with the example. These are modules and should be spelled as the module, probably with some formatting to make it clear that it is a module. |
Sorry, something went wrong.
There was a problem hiding this comment.
Sorry, something went wrong.
|
Thanks for these very quick responses!
For consistency, I've now changed it to The axes_grid1 module , The axisartist module and The pyplot module |
Sorry, something went wrong.
…017-on-v3.7.x Backport PR #25017 on branch v3.7.x (Capitalize headings in example Gallery)
| Back | FazBrowse Home | New Git URL |
The headings in the example section are mostly upper case, but also a few lowercase headings::

This pr capitalizes all headings.