Skip to content

bpo-35555: IDLE: Gray out Code Context menu item on non-editors #11282

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Dec 23, 2018
Merged

bpo-35555: IDLE: Gray out Code Context menu item on non-editors #11282

merged 3 commits into from
Dec 23, 2018

Conversation

csabella
Copy link
Contributor

@csabella csabella commented Dec 21, 2018 •
edited by bedevere-bot
Loading

The Code Context menu item only works on Editor windows, so this will disable the option on windows where it's not available.

https://bugs.python.org/issue35555

@csabella
Copy link
Contributor Author

After looking through the code and checking the usages of EditorWindow and OutputWindow, it appears that disabling the menu option for all Output Windows did the correct thing. There aren't really tests for the menu system yet (PR3669 has some beginnings for menu tests), so this may be best done as a human test for now.

@terryjreedy
Copy link
Member

Looks good. We needed to work you how to best do this for other things as well.

@terryjreedy terryjreedy merged commit 8874f51 into python:master Dec 23, 2018
@bedevere-bot
Copy link

@terryjreedy: Please replace # with GH- in the commit message next time. Thanks!

@csabella csabella deleted the ccdisable branch December 23, 2018 21:38
@miss-islington
Copy link
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

GH-11300 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 23, 2018
@csabella @miss-islington
bpo-35555: IDLE: Gray out Code Context menu item on non-editors (pyth…
c8ede5f 
…onGH-11282)

The Code Context menu item only works on Editor windows so disable it for others.
(cherry picked from commit 8874f51)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
terryjreedy pushed a commit that referenced this pull request Dec 23, 2018
@miss-islington @csabella @terryjreedy
bpo-35555: IDLE: Gray out Code Context menu item on non-editors (GH-1…
d4f7616 
…1282) (GH-11300)

The Code Context menu item only works on Editor windows so disable it for others.
(cherry picked from commit 8874f51)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
terryjreedy
terryjreedy reviewed Mar 12, 2019
View reviewed changes
Misc/NEWS.d/next/macOS/2018-12-21-18-44-30.bpo-35555.M58_K3.rst
@@ -0,0 +1 @@
Gray out Code Context menu entry when it's not applicable.
Copy link
Member

Choose a reason for hiding this comment

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

Putting this in the macOS instead of IDLE section confused Ned into thinking this was a Mac-only change. We should check each other's blurb directories. It is too easy to get the wrong one.

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

@terryjreedy terryjreedy terryjreedy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@csabella @terryjreedy @bedevere-bot @miss-islington @the-knights-who-say-ni