Skip to content

Disable mouseover info for NonUniformImage.#18728

Merged
tacaswell merged 1 commit into
matplotlib:masterfrom
anntzer:nuigcd
Oct 14, 2020
Merged

Disable mouseover info for NonUniformImage.#18728
tacaswell merged 1 commit into
matplotlib:masterfrom
anntzer:nuigcd

Conversation

@anntzer

@anntzer anntzer commented Oct 14, 2020

Copy link
Copy Markdown
Contributor

Currently NonUniformImage just inherits get_cursor_data from the base
class, resulting in nonsensical cursor info. While properly
implementing NonUniformImage.get_cursor_data would be better, at least
we can disable cursor info in the meantime.

PR Summary

PR Checklist

  • Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (run flake8 on changed files to check).
  • New features are documented, with examples if plot related.
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • Conforms to Matplotlib style conventions (install flake8-docstrings and pydocstyle<4 and run flake8 --docstring-convention=all).
  • 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).

Currently NonUniformImage just inherits get_cursor_data from the base
class, resulting in nonsensical cursor info.  While properly
implementing NonUniformImage.get_cursor_data would be better, at least
we can disable cursor info in the meantime.
@timhoffm

Copy link
Copy Markdown
Member

You may want to create a ticket for the mouse over. I think that's a good first issue.

@anntzer

anntzer commented Oct 14, 2020

Copy link
Copy Markdown
Contributor Author

that's already tracked at #18652.

@tacaswell tacaswell merged commit 2d97de2 into matplotlib:master Oct 14, 2020
@tacaswell tacaswell added this to the v3.4.0 milestone Oct 14, 2020
@anntzer anntzer deleted the nuigcd branch October 14, 2020 18:09
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.

3 participants