Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC: clarify how colorbar steals space #25873

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

Merged
merged 3 commits into from
May 15, 2023

Conversation

tacaswell
Copy link
Member

PR summary

In looking at #25871 I noted that the documentation on colorbar was not clear about the default behavior.

The docstring from ax explains the default behavior of the cax parameter and nothing explained the default value of ax.

@tacaswell tacaswell added this to the v3.8.0 milestone May 12, 2023
@tacaswell
Copy link
Member Author

@Wrzlprmft Does this make the expected behavior clearer?

@tacaswell tacaswell closed this May 12, 2023
@tacaswell tacaswell reopened this May 12, 2023
@Wrzlprmft
Copy link

@Wrzlprmft Does this make the expected behavior clearer?

Yes, but as discussed on the original issue, I think that the object structure is the problem, not the documentation.

Co-authored-by: Tim Hoffmann <[email protected]>
Co-authored-by: Jody Klymak <[email protected]>
@tacaswell tacaswell force-pushed the doc/colorbar_spacestealing branch from 2ee6498 to 92d82a9 Compare May 14, 2023 20:11
Co-authored-by: Tim Hoffmann <[email protected]>
@timhoffm timhoffm merged commit dbe7c2d into matplotlib:main May 15, 2023
@tacaswell tacaswell deleted the doc/colorbar_spacestealing branch June 29, 2023 03:31
@tacaswell tacaswell modified the milestones: v3.8.0, v3.7.2 Jun 29, 2023
@tacaswell
Copy link
Member Author

tacaswell commented Jun 29, 2023

@ backport to 3.7.x I did it "by hand"

tacaswell pushed a commit to tacaswell/matplotlib that referenced this pull request Jun 29, 2023
…stealing

DOC: clarify how colorbar steals space
(cherry picked from commit dbe7c2d)
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.

5 participants