Skip to content

Conversation

oscargus
Copy link
Member

@oscargus oscargus commented Jan 5, 2022

PR Summary

Closes #22012

Not clear how to add a test for it.

Also, should this be documented with a "what's new"? Seems quite marginal...

(I couldn't figure out how the macosx-version worked, and do not have a mac to test it...)

PR Checklist

Tests and Styling

  • Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (install flake8-docstrings and run flake8 --docstring-convention=all).

Documentation

  • New features are documented, with examples if plot related.
  • New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
  • API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).

Copy link
Contributor

@anntzer anntzer left a comment

Choose a reason for hiding this comment

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

lgtm.
I don't think this is particularly testable (other than manually).

@dstansby dstansby added this to the v3.6.0 milestone Jan 5, 2022
@dstansby dstansby merged commit 7549f8c into matplotlib:main Jan 5, 2022
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]: Mouseover coordinate/value text should be right aligned
3 participants