Skip to content

Conversation

@greglucas
Copy link
Contributor

PR Summary

This makes the images used for the buttons a template image which can be adjusted based on the system theme chosen by the user. The text in the toolbar when hovering an image already follows the theme changing from white/black appropriately.

Light mode:
image

Dark mode:
image

Closes #22605

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).

This makes the images used for the buttons a template image which
can be adjusted based on the system theme chosen by the user.
Copy link
Member

@dstansby dstansby left a comment

Choose a reason for hiding this comment

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

Awesome!

@dstansby dstansby added this to the v3.6.0 milestone Mar 5, 2022
@timhoffm timhoffm merged commit fbfcacc into matplotlib:main Mar 5, 2022
@greglucas greglucas deleted the macosx-button-theme branch March 5, 2022 23:53
@QuLogic QuLogic mentioned this pull request Sep 9, 2022
2 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.

[Bug]: Tool contrast low with dark theme on macosx backend

3 participants