Skip to content

Skip displaying pan/zoom navigate mode in toolbar. #24869

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
Jan 6, 2023

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Jan 2, 2023

It doesn't add much over the button state and the button tooltip (note that the state isn't even displayed when the cursor is over an axes, as the coordinates take precedence), and collides with other possible end-user customizations of the toolbar (currently, end-users can override the coordinate string via format_coord, but not the navigate mode string).

Closes #24867.

PR Summary

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • New plotting related features are documented with examples.

Release Notes

  • New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

It doesn't add much over the button state and the button tooltip (note
that the state isn't even displayed when the cursor is over an axes, as
the coordinates take precedence), and collides with other possible
end-user customizations of the toolbar (currently, end-users can
override the coordinate string via format_coord, but not the navigate
mode string).
@timhoffm
Copy link
Member

timhoffm commented Jan 3, 2023

Which milestone should this go to?

@anntzer
Copy link
Contributor Author

anntzer commented Jan 3, 2023

I don't really care either way.

@oscargus oscargus added this to the v3.7.0 milestone Jan 6, 2023
@oscargus oscargus merged commit 439f0ff into matplotlib:main Jan 6, 2023
@anntzer anntzer deleted the um branch January 6, 2023 13:24
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.

[Bug]: controlling text on toolbar in wx
4 participants