Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-39388: IDLE: Fix bug when cancelling out of configdialog #18068

Merged
merged 5 commits into from Jan 25, 2020

Conversation

csabella
Copy link
Contributor

@csabella csabella commented Jan 19, 2020

Changes to keybindings were being remembered even when cancelling out of the Config Dialog.

https://bugs.python.org/issue39388

@csabella csabella requested a review from terryjreedy as a code owner Jan 19, 2020
@csabella csabella changed the title bpo-39388: Fix bug when cancelling out of configdialog bpo-39388: IDLE: Fix bug when cancelling out of configdialog Jan 19, 2020
Copy link
Member

@terryjreedy terryjreedy left a comment

A 2nd button bug not caught by missing tests. We wrote changes.clear to be called at this spot. (It is not called anywhere else.) I will add a test that changes is cleared and add news items.

@terryjreedy terryjreedy merged commit d0d9fa8 into python:master Jan 25, 2020
9 checks passed
@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jan 25, 2020

Thanks @csabella for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8.
🐍🍒🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jan 25, 2020
…H-18068)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit d0d9fa8)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Jan 25, 2020

GH-18178 is a backport of this pull request to the 3.8 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jan 25, 2020
…H-18068)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit d0d9fa8)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Jan 25, 2020

GH-18179 is a backport of this pull request to the 3.7 branch.

sthagen added a commit to sthagen/python-cpython that referenced this issue Jan 25, 2020
bpo-39388: IDLE: Fix bug when cancelling out of configdialog (pythonGH-18068)
miss-islington added a commit that referenced this issue Jan 25, 2020
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit d0d9fa8)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
miss-islington added a commit that referenced this issue Jan 25, 2020
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit d0d9fa8)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
@csabella
Copy link
Contributor Author

@csabella csabella commented Jan 25, 2020

I had some additional test for configdialog in GH-3592 and I rebased it last week. It's not all the missing tests.

@csabella csabella deleted the cancelconfig branch Jan 25, 2020
shihai1991 pushed a commit to shihai1991/cpython that referenced this issue Jan 31, 2020
opavlyuk pushed a commit to opavlyuk/cpython that referenced this issue Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA signed type-bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants