Skip to content

ENH: support x/y-axis zoom#30543

Merged
QuLogic merged 3 commits into
matplotlib:mainfrom
LangQi99:feat/zoom
Oct 2, 2025
Merged

ENH: support x/y-axis zoom#30543
QuLogic merged 3 commits into
matplotlib:mainfrom
LangQi99:feat/zoom

Conversation

@LangQi99

@LangQi99 LangQi99 commented Sep 10, 2025

Copy link
Copy Markdown
Contributor

PR summary

Closes #30541

Ctrl+Wheel zoom with fixed axes ratio
y+Wheelzoom y
x+Wheelzoom x

PR checklist

@LangQi99 LangQi99 changed the title feat: support x/y-axis zoom ENH: support x/y-axis zoom Sep 10, 2025
@LangQi99 LangQi99 marked this pull request as ready for review September 10, 2025 14:59

@timhoffm timhoffm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Overall, I think this is the right approach, but let's see where the discussion in #30541 lsands.

Comment thread lib/matplotlib/backend_bases.py Outdated
Comment thread lib/matplotlib/backend_bases.py Outdated
@tacaswell tacaswell added this to the v3.11.0 milestone Sep 11, 2025
LangQi99 and others added 2 commits September 12, 2025 11:21
"X" would be generated by "Shift+x"

Co-authored-by: Tim Hoffmann <[email protected]>
"X" would be generated by "Shift+x"

Co-authored-by: Tim Hoffmann <[email protected]>
@LangQi99

Copy link
Copy Markdown
Contributor Author

Thanks! @timhoffm

@LangQi99

LangQi99 commented Sep 12, 2025

Copy link
Copy Markdown
Contributor Author

I noticed that some new CI checks are now failing.

https://github.com/matplotlib/matplotlib/pull/30543/commits/0a744f0ba180dca0a73bf351db89796a9854c7d6

I was wondering if this is an expected outcome, or if there's something new I need to adjust on my end? Any guidance would be much appreciated!

@timhoffm

timhoffm commented Sep 12, 2025

Copy link
Copy Markdown
Member

The stubtest failure is unrelated to this PR. - Likely a change in mypy, which recently released v1.18 - That can be handled independently (opened #30551 for that).

@QuLogic QuLogic merged commit 1c95f1b into matplotlib:main Oct 2, 2025
63 of 66 checks passed
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.

[ENH]: Zooming single axis

4 participants