Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Add styling support to Check and Radio buttons #24838

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
ksunden merged 4 commits into matplotlib:main from QuLogic:button-props
Jan 23, 2023

Conversation

Copy link
Member

@QuLogic QuLogic commented Dec 29, 2022 •
edited
Loading

PR Summary

This implements arguments to override the styling of CheckButtons and RadioButtons, in a manner similar to the existing ToolHandles or Selectors. This depends on #23457 to avoid the conflicts on the arguments. I updated the examples to show what this looks like, but not the tests or documentation yet. I have two questions:

  1. Are the argument names reasonable?
  2. What to do about the label properties? The other items are put in scatter, which does have a Collection, so they accept dict with elements that are singular or lists. However, we don't have a TextCollection, so I had to make it take a dict or list of dict. This seems inconsistent, so perhaps I should attempt to implement the former input here?

Fixes #24583

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

actives=[l.get_visible() for l in lines_by_label.values()],
label_props=[{'color': 'k'}, {'color': 'r'}, {'color': 'g'}],
frame_props={'edgecolor': ['k', 'r', 'g']},
check_props={'facecolor': ['k', 'r', 'g']},
Copy link

@drammock drammock Jan 5, 2023

Choose a reason for hiding this comment

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

I looked at the rendered doc and noticed that the Xs are still black, I think maybe edgecolor is what was intended here?

Suggested change
check_props={'facecolor': ['k', 'r', 'g']},
check_props={'edgecolor': ['k', 'r', 'g']},

Copy link
Member Author

@QuLogic QuLogic Jan 5, 2023

Choose a reason for hiding this comment

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

No, the bug is in widgets.py; I just need to work out what exactly the semantics for color/facecolor/edgecolor will be.

drammock reacted with thumbs up emoji
Copy link
Member

ksunden commented Jan 17, 2023

@QuLogic I think you had indicated in alternate communication channels that this was ready for review, but it is still marked as draft.

Copy link
Member Author

QuLogic commented Jan 17, 2023

@ksunden Ah, sorry, I thought I pushed the button here; maybe an outdated browser page or something.

@QuLogic QuLogic marked this pull request as ready for review January 17, 2023 01:07
Copy link
Member

ksunden commented Jan 17, 2023

Probably should have a whatsnew entry to highlight the api changes

Copy link
Member Author

QuLogic commented Jan 19, 2023

Added the what's new and handling of activecolor as mentioned in call yesterday.

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.

Lots of suggestions and minor comments. The major change request I have is to double check this is working - I think it's worth adding a test that doesn't compare new API to new API, and looking at the what's new entry some of the styling for check buttons doesn't seem to be applied.

Copy link
Member Author

QuLogic commented Jan 21, 2023

I modified a test image to include old and new versions, and I think should've handling your other comments in a previous push.

@ksunden ksunden merged commit bf2d1dc into matplotlib:main Jan 23, 2023
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jan 23, 2023
ksunden added a commit that referenced this pull request Jan 23, 2023 
...838-on-v3.7.x
Backport PR #24838 on branch v3.7.x (Add styling support to Check and Radio buttons )
@QuLogic QuLogic deleted the button-props branch January 23, 2023 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tacaswell tacaswell tacaswell left review comments

@ksunden ksunden ksunden left review comments

@dstansby dstansby dstansby approved these changes

+1 more reviewer

@drammock drammock drammock left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Projects
None yet
Milestone
v3.7.0
Development

Successfully merging this pull request may close these issues.

[ENH]: provide public API for styling radio buttons

AltStyle によって変換されたページ (->オリジナル) /