Skip to content

gh-157762: Fix Alt-2 zoom height binding in IDLE tree windows - #157772

Open
lazerg wants to merge 1 commit into
python:mainfrom
lazerg:fix-issue-157762
Open

lazerg wants to merge 1 commit into
python:mainfrom
lazerg:fix-issue-157762

Conversation

@lazerg

@lazerg lazerg commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

ScrolledCanvas binds Alt-2 to zoom_height, which called idlelib.zoomheight.zoom_height. That module-level function went away with the Zoom Height rework in 2019, so Alt-2 in Path Browser, Module Browser or the stack viewer raises AttributeError out of the Tk callback.

ZoomHeight only needs the toplevel of the window it zooms, so ScrolledCanvas keeps one and calls ZoomHeight(self).zoom_height().

Fixes #157762

@terryjreedy

Copy link
Copy Markdown
Member

Lazizbek: thank you for the PR. For reasons given on the issue, I am putting the issue and PR on hold.

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.

IDLE: ScrolledCanvas.zoom_height calls idlelib.zoomheight.zoom_height, which has not existed since 2019

2 participants