Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
WIP: Add support to edit subplot configurations via textbox#19271
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@aitikgupta I've converted this to "draft". This signals reviewers that they don't yet have to look at it. When you think this is ready for a closer look (doesn't have to be 100% completed), you can click the "Ready for review" button. And of course, ping us whenever there is a question. |
We may have talked past eachother a little bit. In that issue I was proposing that this be a feature intrinsic to all sliders, not just the ones in the subplots adjust panel. So this may well be a valuable addition, but it shouldn't close that issue. Of course if this was implemented for every slider then you would get it for free in the subplot panel. |
That actually makes more sense! Ah, that's why you were talking about |
Since this Pull Request has not been updated in 60 days, it has been marked "inactive." This does not mean that it will be closed, though it may be moved to a "Draft" state. This helps maintainers prioritize their reviewing efforts. You can pick the PR back up anytime - please ping us if you need a review or guidance to move the PR forward! If you do not plan on continuing the work, please let us know so that we can either find someone to take the PR over, or close it. |
Uh oh!
There was an error while loading.Please reload this page.
PR Summary
This PR adds support to manually edit the slider values in the subplot configuration tool via a TextBox.
(potentially a
NumericTextBox
)This is a WIP, to mockup the desired behavior mentioned at#19257 (comment).
Behaviour:

To test it out, you can use any snippet:
A partial fix to#19257
PR Checklist
pytest
passes).flake8
on changed files to check).flake8-docstrings
and runflake8 --docstring-convention=all
).doc/users/next_whats_new/
(follow instructions in README.rst there).doc/api/next_api_changes/
(follow instructions in README.rst there).