Skip to content

Fix: Prevent Cursor blitting from erasing overlapping axes (#25670)#31368

Open
MohitPal2005 wants to merge 2 commits intomatplotlib:mainfrom
MohitPal2005:fix-blitting-25670
Open

Fix: Prevent Cursor blitting from erasing overlapping axes (#25670)#31368
MohitPal2005 wants to merge 2 commits intomatplotlib:mainfrom
MohitPal2005:fix-blitting-25670

Conversation

@MohitPal2005
Copy link

Fixes #25670.

Bug: When Cursor is used with useblit=True, its background restore operation blindly overwrites any overlapping widgets (like CheckButtons residing in a different axes on the same figure), causing them to flicker or disappear.

Fix: Updated Cursor.onmove to explicitly detect if any other axes overlap with the cursor's main axes bounding box. If an overlap is detected, the overlapping axes are explicitly redrawn before blitting to the canvas, preserving their state. This serves as a localized fix for widget interaction prior to a larger architectural overlay manager.

@github-actions
Copy link

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks. We also ask that you please finish addressing any review comments on this PR and wait for it to be merged (or closed) before opening a new one, as it can be a valuable learning experience to go through the review process.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide.
Please let us know if (and how) you use AI, it will help us give you better feedback on your PR.

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@MohitPal2005
Copy link
Author

Hi team, I pushed the fixes for the ruff styling checks.

I noticed the docs-python3 CircleCI check is failing on the environment setup ([Errno 2] No such file or directory: 'requirements/dev/build-requirements.txt'). Since my changes are strictly isolated to lib/matplotlib/widgets.py, I suspect this might be an unrelated upstream CI issue.

Please let me know if there is anything specific you need me to adjust on my end!

@melissawm melissawm moved this to Needs review in First Time Contributors Mar 25, 2026
@melissawm
Copy link
Member

CircleCI is showing a degraded status banner right now so it could be a transient issue. Maybe we can re-trigger the build later.

@MohitPal2005
Copy link
Author

Thanks @melissawm!

Just as a heads up, the Main Pytest Windows_py313 check also failed. Looking at the logs, it appears to be due to a subprocess.TimeoutExpired in test_interactive_thread_safety and test_lazy_auto_backend_selection. Since both tests are marked as flaky and involve interactive backends, this seems like an unrelated CI timeout rather than anything caused by my changes in widgets.py.

Please let me know if you would like me to investigate further or make any adjustments. Happy to help!

@melissawm
Copy link
Member

Looking at this more carefully, it looks like you may need to rebase to main, since the circleci action should not be looking for a requirements file anymore:

python -m pip install --user \

@MohitPal2005
Copy link
Author

Hi @melissawm, the rebase was successful!

However, it looks like the Main Pytest Windows and Python 3.13t checks are failing catastrophically now (hitting the 100-failure abort limit instantly). Since my changes are strictly scoped to the Cursor blitting logic in widgets.py, I assume this is a larger upstream environment issue with those specific CI runners. Let me know if you need me to do anything else!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Needs review

Development

Successfully merging this pull request may close these issues.

[Bug]: Cursor blitting produces CheckButton issue (backend-agnostic)

2 participants