Skip to content

gh-71807: The escape key closes IDLE's config dialog as canceled#19706

Open
ZackerySpytz wants to merge 2 commits into
python:mainfrom
ZackerySpytz:bpo-27620-IDLE-escape-key-config
Open

gh-71807: The escape key closes IDLE's config dialog as canceled#19706
ZackerySpytz wants to merge 2 commits into
python:mainfrom
ZackerySpytz:bpo-27620-IDLE-escape-key-config

Conversation

@ZackerySpytz

@ZackerySpytz ZackerySpytz commented Apr 25, 2020

Copy link
Copy Markdown
Contributor

@ZackerySpytz

Copy link
Copy Markdown
Contributor Author

I think the escape key binding should be enabled, but I'm not so sure about the other ones.

@terryjreedy terryjreedy changed the title bpo-27620: The escape key closes IDLE's config dialog as canceled gh-71807: The escape key closes IDLE's config dialog as canceled Sep 18, 2022

@terryjreedy terryjreedy 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.

PR title and blurb only refer to Escape to exit (cancel). I may limit PR to this, but expand to subsidiary dialogs that are not query boxes. (Should check code since it works.)

If there is a key for [Apply], there should be a key for [OK].
Ctrl-Enter and Alt-Enter might be a good pair.

Tab navigation has multiple problems: Never gets to these buttons.
Tabbing into font sample traps focus on the text widget.
Some widgets get skipped.

Other key bindings are needed: No way to toggle button or select radio button.

Changes are needed, but I am not sure which to do at the moment.

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants