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

Skip to content

DOC: Cleanup spines usage in examples #24378

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 1 commit into from
Nov 6, 2022

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Nov 6, 2022

  • hide spines using set_visible(False) instead of set_color('none')
  • remove ax.xaxis.set_ticks_position('bottom') and ax.yaxis.set_ticks_position('left'), because these are already the defaults
  • use multi-assignment (spines[['left', 'bottom']].set_*) when possible
  • Remove "Custom spine bounds" example. This is already included in the "Spines" example

- hide spines using `set_visible(False)` instead of `set_color('none')`
- remove `ax.xaxis.set_ticks_position('bottom')` and
  `ax.yaxis.set_ticks_position('left')`, because these are already the
  defaults
- use multi-assignment (`spines[['left', 'bottom']].set_*`) when
  possible
- Remove "Custom spine bounds" example. This is already included in the
  "Spines" example
@timhoffm timhoffm added this to the v3.6.3 milestone Nov 6, 2022
@tacaswell
Copy link
Member

@timhoffm did you mean v3.6.3 instead of v3.6-doc?

oscargus added a commit that referenced this pull request Nov 6, 2022
…378-on-v3.6.x

Backport PR #24378 on branch v3.6.x (DOC: Cleanup spines usage in examples)
@timhoffm
Copy link
Member Author

timhoffm commented Nov 6, 2022

@timhoffm did you mean v3.6.3 instead of v3.6-doc?

I'm confused. This milestoned v3.6.3 as suggested.

@timhoffm timhoffm deleted the doc-spines-cleanup branch November 6, 2022 09:14
@ksunden ksunden mentioned this pull request Feb 20, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants