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-31206: IDLE: Factor HighPage class from ConfigDialog #3141

Merged
merged 1 commit into from
Aug 18, 2017

Conversation

csabella
Copy link
Contributor

@csabella csabella commented Aug 18, 2017

I didn't know if the blurb should be on this PR or the second one.

https://bugs.python.org/issue31206

Copy link
Member

@terryjreedy terryjreedy left a comment

Choose a reason for hiding this comment

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

Blurb could have waited, but now is okay.

@terryjreedy terryjreedy merged commit a32e405 into python:master Aug 18, 2017
@csabella csabella deleted the bpo31206 branch August 19, 2017 00:35
csabella added a commit to csabella/cpython that referenced this pull request Aug 19, 2017
…nGH-3141)

This is the first half of a patch similar to the one for for bpo-31205.  It is being split into 2 PRs to avoid what happened with PR-3096 -- an incomprehensible diff that could not be cleanly backported to 3.6.  This half copies several methods of ConfigDialog and turns them into a new class.
(cherry picked from commit a32e405)
@bedevere-bot
Copy link

GH-3154 is a backport of this pull request to the 3.6 branch.

terryjreedy pushed a commit that referenced this pull request Aug 19, 2017
…) (#3154)

This is the first half of a patch similar to the one for for bpo-31205.  It is being split into 2 PRs to avoid what happened with PR-3096 -- an incomprehensible diff that could not be cleanly backported to 3.6.  This half copies several methods of ConfigDialog and turns them into a new class.  Patch by Cheryl Sabella.
(cherry picked from commit a32e405)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants