Addsgrid.minor.* params tomatplotlibrc allowing one to set different parameters for minor grids. I've been manually turning on minor ticks with.grid(which="minor"), but this gets tiring and I'd like to be able to just enable this globally in thematplotlibrc file. However, both themajor andminor parameters are currently configured withgrid.*, with no distinction forminor ormajor. The present PR attempts to address this.

I can add examples / test / docs etc. if this might be an acceptable change.

PR checklist

[N/A] "closes #0000" is in the body of the PR description tolink the related issue
new and changed code istested
Plotting related features are demonstrated in anexample
New Features andAPI Changes are noted with adirective and release note
Documentation complies withgeneral anddocstring guidelines

Allow separate config of minor grid params in matplotlibrc.

06665bf

github-actionsbot reviewed

Jun 14, 2023

View reviewed changes

Copy link

github-actionsbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join uson gitter for real-time discussion.

For details on testing, writing docs, and our review process, please seethe developer guide

We strive to be a welcoming and open project. Please follow ourCode of Conduct.

melissawm added the topic: ticks axis labels label

Jun 15, 2023

github-actionsbot added the status: needs rebase label

Jul 28, 2023

Copy link

Author

RJTK commentedAug 5, 2023

Is there interest in this?

Copy link

Member

jklymak commentedAug 5, 2023

It seems ok but is a breaking change for any style that customized the grid style and expected the minor grid to change as well?

Copy link

Member

QuLogic commentedAug 5, 2023

Yes, thegrid.minor.* rcParams should probably default to something (name to be determined) that indicates it should fall back to the original major grid'sgrid.* rcParams version.

Copy link

Member

QuLogic commentedAug 5, 2023

cc@timhoffm on that.

Copy link

Author

RJTK commentedAug 6, 2023

Yes, the grid.minor.* rcParams should probably default to something (name to be determined) that indicates it should fall back to the original major grid's grid.* rcParams version.

Agreed, I can update to ignore the .minor.* param unless explicitly set.

Copy link

Member

timhoffm commentedAug 7, 2023

I think the pragmatic approach here is do use "inherit" as a default value and if that is encountered, use the respective non-minor grid value.

This also rhymes with the layering hierarchy that (major) is undergrid.* and minor is one level below undergrid.minor.*.

A revised config framework may add reusing other values within the config classes as well, but for now, the pragmatic solution is to hard-code that inheritance at the usage sites ofgrid.minor.