Skip to content

bpo-31206: IDLE: Factor HighPage class from ConfigDialog#3141

Merged
terryjreedy merged 1 commit into
python:masterfrom
csabella:bpo31206
Aug 18, 2017
Merged

bpo-31206: IDLE: Factor HighPage class from ConfigDialog#3141
terryjreedy merged 1 commit into
python:masterfrom
csabella:bpo31206

Conversation

@csabella
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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