◐ Shell
reader mode source ↗
Skip to content

bpo-31050: IDLE: Factor GenPage class from ConfigDialog#2952

Merged
terryjreedy merged 2 commits into
python:masterfrom
csabella:bpo31050
Jul 30, 2017
Merged

bpo-31050: IDLE: Factor GenPage class from ConfigDialog#2952
terryjreedy merged 2 commits into
python:masterfrom
csabella:bpo31050

Conversation

@csabella

@csabella csabella commented Jul 30, 2017

Copy link
Copy Markdown
Contributor

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

Hide comment

Add blurb with
bpo-31004: Factor FontPage(Frame) class from ConfigDialog.
The slightly modified tests continue to pass.
Patch by Cheryl Sabella.

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

Hide comment

I tested each widget on the page and checked that the corresponding change had been made in my user config-main. (Automating this would be the subject of a #24845 spinoff.)

@terryjreedy terryjreedy changed the title bpo-31050: IDLE: Factor out GenTab class from ConfigDialog Jul 30, 2017
@terryjreedy terryjreedy merged commit e8eb17b into python:master Jul 30, 2017
terryjreedy pushed a commit to terryjreedy/cpython that referenced this pull request Jul 30, 2017
…GH-2952)

The slightly modified tests for the General tab continue to pass.
Patch by Cheryl Sabella.
(cherry picked from commit e8eb17b)
terryjreedy added a commit that referenced this pull request Jul 30, 2017
… (#2955)

The slightly modified tests for the General tab continue to pass.
Patch by Cheryl Sabella.
(cherry picked from commit e8eb17b)
@csabella csabella deleted the bpo31050 branch July 31, 2017 00:35
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.

3 participants