Skip to content

bpo-35555: Create function to update IDLE menu item state#11343

Merged
terryjreedy merged 1 commit intopython:masterfrom
csabella:bpo35555
Dec 28, 2018
Merged

bpo-35555: Create function to update IDLE menu item state#11343
terryjreedy merged 1 commit intopython:masterfrom
csabella:bpo35555

Conversation

@csabella
Copy link
Copy Markdown
Contributor

@csabella csabella commented Dec 28, 2018

  • Instead of updating menu state directly, create function to update the state.
  • Change outwin to call new function instead of updating menu directly.

https://bugs.python.org/issue35555

* Instead of updating menu state directly, create function
  to update the state.
* Change outwin to call function instead of updating menu
  directly.
Copy link
Copy Markdown
Contributor

@eamanu eamanu left a comment

Choose a reason for hiding this comment

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

Seem a good improve on the code.

@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @csabella for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@bedevere-bot
Copy link
Copy Markdown

GH-11344 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 28, 2018
This will be needed for other menu items.  Change outwin to call the function instead of updating the menu item directly.
(cherry picked from commit 804a5d9)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
@terryjreedy terryjreedy changed the title bpo-35555: IDLE: Create function to update menu item state bpo-35555: Create function to update IDLE menu item state Dec 28, 2018
miss-islington added a commit that referenced this pull request Dec 28, 2018
This will be needed for other menu items.  Change outwin to call the function instead of updating the menu item directly.
(cherry picked from commit 804a5d9)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants