Home Original page

bpo-27755: IDLE: Convert configdialog DynOptionMenu to ttk OptionMenu by csabella · Pull Request #3215 · python/cpython

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@csabella

Copy link

Contributor

@csabella csabella commented

Aug 26, 2017

•

edited by bedevere-bot

Loading

I know you may want to get rid of Option Menus, but I wanted to show you the ttk version. This was too much of a change to include with the first patch.

https://bugs.python.org/issue30781

@terryjreedy

Copy link

Member

This is a definite improvement. A single OptionMenu could work for a unified list of keysets. For themes, I am thinking of a listbox, as with fonts, so one can move around to see the effect of different choices without having the box close.

@terryjreedy terryjreedy changed the title bpo-30781: IDLE: Convert configdialog DynOptionMenu to ttk OptionMenu bpo-27755: IDLE: Convert configdialog DynOptionMenu to ttk OptionMenu

Sep 11, 2017

@csabella csabella closed this

Sep 23, 2018

@terryjreedy

Copy link

Member

Why? The change is still needed.

@csabella

Copy link

Contributor Author

I was reading through the bug tracker and it looked like Spinbox and/or Combobox would be used instead.

https://bugs.python.org/issue33962

@terryjreedy

Copy link

Member

Right. I found where I rejected this in its current form, and added more on current uses and replacements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@csabella @terryjreedy @the-knights-who-say-ni @bedevere-bot