classification
Title: IDLE: Re-order run menu
Type: enhancement Stage: resolved
Components: Versions: Python 3.9, Python 3.8, Python 3.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: cheryl.sabella, miss-islington, rhettinger, taleinat, terry.reedy
Priority: normal Keywords: patch

Created on 2019-08-03 04:48 by terry.reedy, last changed 2019-08-04 21:46 by terry.reedy. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 15115 merged terry.reedy, 2019-08-04 19:54
PR 15119 merged miss-islington, 2019-08-04 20:45
PR 15120 merged miss-islington, 2019-08-04 20:45
Messages (8)
msg348940 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2019-08-03 04:48
With the addition of Run Customized, the run menu looks like

Python Shell
Check Module
Run Module
Run... Customized

This order resulted from Check and Run Module originally being implemented as extensions, which forced them to be below Python Shell.  The situation was similar with Format Paragraph being forced below less commonly used options on the Format menu.  It is now at the top.
Run... Customized was tacked on below, without worrying about the appropriate order for the menu..

On issue #37627, Raymond Hettinger suggested "Move the [new] menu option up by one so that the regular F5 "run" is last -- learners we having a hard time finding and mouse targeting the more commonly used regular "run" option.  The result would be

Python Shell
Check Module
Run... Customized
Run Module

I would rather make Run Module, the most commonly used option even more prominent by putting it at the top, perhaps by reversing the order.

Run Module
Run... Customized
Check Module
Python Shell

With Run Module at the top, remaining items could be ordered by logic, and to me, the above is plausible on that score, or by use, perhaps putting the second-most used at the bottom.  I suspect that this will be Run Customized.  To me, Check Module is useless, though I can imagine situation when it is not.  Python Shell is only needed when there is no shell and one does not want to make one appear by running a module.  To switch to an existing Shell, on should use the Windows menu.  Anyway, another candidate is

Run Module
Check Module
Python Shell
Run... Customized
msg348958 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2019-08-03 13:11
Perhaps we should use a separator to differentiate between things that run in the shell and those that don't, e.g. as following:

Run Module
Run... Customized
Python Shell
-------------
Check Module
msg348966 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2019-08-03 17:45
Run Module and Run Customized both run Check Module first.  All three involve the code in the particular editor.  Restarting Shell is not editor specific, so I see *it* as the oddball.

The proposed Run Checker #21880, would involve the editor code, might or might not Check Module first, but would not run in or involve Shell at all.  It might be a good candidate for the last position.  If it were added, I think my first option, Run Module ... Python Shell would be better.
msg348967 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2019-08-03 20:07
Ah, my bad, I mixed up "Check Module" with the proposed external checkers feature.

Anyways, I'm +1 on this reordering, but have no opinion on the specifics.
msg348972 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2019-08-04 02:50
+1 for the proposed reverse ordering:

Run Module
Run... Customized
Check Module
Python Shell
msg349003 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2019-08-04 20:45
New changeset 14070299cdc0faf36975f0cc2d51824a9abf3db0 by Terry Jan Reedy in branch 'master':
bpo-37748: Re-order the Run menu. (GH-15115)
https://github.com/python/cpython/commit/14070299cdc0faf36975f0cc2d51824a9abf3db0
msg349005 - (view) Author: miss-islington (miss-islington) Date: 2019-08-04 21:04
New changeset 9c95fc752c1465202df67fa894ef326c8ebb8cac by Miss Islington (bot) in branch '3.8':
bpo-37748: Re-order the Run menu. (GH-15115)
https://github.com/python/cpython/commit/9c95fc752c1465202df67fa894ef326c8ebb8cac
msg349006 - (view) Author: miss-islington (miss-islington) Date: 2019-08-04 21:07
New changeset a96f0367d4c84ed42f8dc80c88c10f334498f36c by Miss Islington (bot) in branch '3.7':
bpo-37748: Re-order the Run menu. (GH-15115)
https://github.com/python/cpython/commit/a96f0367d4c84ed42f8dc80c88c10f334498f36c
History
Date User Action Args
2019-08-04 21:46:11terry.reedysetstatus: open -> closed
resolution: fixed
stage: patch review -> resolved
2019-08-04 21:07:28miss-islingtonsetmessages: + msg349006
2019-08-04 21:04:53miss-islingtonsetnosy: + miss-islington
messages: + msg349005
2019-08-04 20:45:33miss-islingtonsetpull_requests: + pull_request14860
2019-08-04 20:45:26miss-islingtonsetpull_requests: + pull_request14859
2019-08-04 20:45:18terry.reedysetmessages: + msg349003
2019-08-04 19:54:22terry.reedysetkeywords: + patch
stage: needs patch -> patch review
pull_requests: + pull_request14856
2019-08-04 02:50:43rhettingersetmessages: + msg348972
2019-08-03 20:07:24taleinatsetmessages: + msg348967
2019-08-03 17:45:36terry.reedysetmessages: + msg348966
2019-08-03 13:12:00taleinatsetmessages: + msg348958
2019-08-03 04:52:50terry.reedylinkissue37627 superseder
2019-08-03 04:48:25terry.reedycreate