bpo-27755: IDLE: Convert configdialog DynOptionMenu to ttk OptionMenu by csabella · Pull Request #3215 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.7k
Conversation
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.
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
changed the title
bpo-30781: IDLE: Convert configdialog DynOptionMenu to ttk OptionMenu
bpo-27755: IDLE: Convert configdialog DynOptionMenu to ttk OptionMenu
Right. I found where I rejected this in its current form, and added more on current uses and replacements.
jfoo
mannequin
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters