Fix: Prevent Cursor blitting from erasing overlapping axes (#25670)#31368
Fix: Prevent Cursor blitting from erasing overlapping axes (#25670)#31368MohitPal2005 wants to merge 2 commits intomatplotlib:mainfrom
Conversation
|
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. We strive to be a welcoming and open project. Please follow our Code of Conduct. |
|
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! |
|
CircleCI is showing a degraded status banner right now so it could be a transient issue. Maybe we can re-trigger the build later. |
|
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 Please let me know if you would like me to investigate further or make any adjustments. Happy to help! |
|
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: matplotlib/.circleci/config.yml Line 107 in a85564e |
1c2f132 to
0e0b72a
Compare
|
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! |
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.