msg152050 - (view) |
Author: Tim McGreevy (mcgrete) |
Date: 2012-01-26 23:41 |
When selecting from menu:
File --> Recent Files
the 'Recent Files' dropdown list undocks from the IDLE gui / File dropdown list. Even after selecting a past file, it remains open until closed manually.
Ubuntu LUCID
amd64
IDLE 2.6.5
TK version 8.5
All installed using Ubuntu supported PPA (Synaptic)
After closing 'Recent Files' subwindow, problem no longer persists until IDLE is terminated and restarted.
|
msg152054 - (view) |
Author: Ezio Melotti (ezio.melotti) *  |
Date: 2012-01-26 23:58 |
Could you try with IDLE 2.7/3.2?
|
msg152067 - (view) |
Author: Terry J. Reedy (terry.reedy) *  |
Date: 2012-01-27 06:45 |
IDLE has tear-off menus. From Help/IDLE Help:
"Click on the dotted line at the top of a menu to "tear it off":
a separate window containing the menu is created."
This is a feature, not a bug.
On 3.2.2, Win7, the Recent Files sub-menu cannot be torn off unless and until the File menu is torn off.
|
msg152192 - (view) |
Author: Roger Serwy (roger.serwy) *  |
Date: 2012-01-28 21:54 |
Should tear-off menus still be a feature? This relates to the #13504 meta-issue, in particular entry "4) ANNOYANCE: Get rid of the detachable menus feature.."
Attached is patch against 3.3a0 to disable tear-off menus. (It also makes modifications to Mac-specific code, but I can't test it since I don't have a Mac.)
|
msg173284 - (view) |
Author: Terry J. Reedy (terry.reedy) *  |
Date: 2012-10-18 17:04 |
I wonder if anyone uses this feature and would miss it if gone. Since it is a simple matter of passing 'tearoff=0/1' on menu creation, perhaps it should be a configuration feature.
|
msg173291 - (view) |
Author: Todd Rovito (Todd.Rovito) * |
Date: 2012-10-18 19:54 |
Terry,
I would not miss the feature at all this is very old school Unix like and most people find it confusing. At my job I tend to provide Python support to my co-workers and have had many questions about it. In addition I think #13504 meta issue provides even more evidence that removing the tear off menus would be a good thing. Something to consider with IDLE is it is often the first thing a user sees when using Python and first impressions count in most circumstances. IDLE could be much improved and Roger Serwy has a good start on improvements with his IdleX project. Thanks for the review and I for one would appreciate this patch getting committed.
|
msg173293 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) *  |
Date: 2012-10-18 20:12 |
Yes, please, make it a configuration feature (off by default). Tear-off menus useful for large and deeply nested menus. Do not forget about people with disabilities.
|
msg173300 - (view) |
Author: Ned Deily (ned.deily) *  |
Date: 2012-10-18 21:53 |
Keep in mind that tear-off menus are not supported on IDLE on OS X when using Aqua Tk (either Carbon or Cocoa), the standard these days. That's primarily because Aqua Tk uses native OS X menu bars which appear once at the top of the desktop, not in each window. There are no dotted line menu bars there. The desktop menu bar menu items change to reflect which window currently has keyboard focus.
|
msg230381 - (view) |
Author: Ezio Melotti (ezio.melotti) *  |
Date: 2014-10-31 19:27 |
I haven't seen any other program using it, so I wouldn't mind if they were removed. People with disabilities probably have better ways to deal with menus, but if adding the option is trivial it's probably better to add it.
|
msg247611 - (view) |
Author: Mark Roseman (markroseman) * |
Date: 2015-07-29 19:42 |
As indicated in prior comments, the tearoff menus are strictly a holdover from ancient Motif, and are no longer found in current user interfaces on any platform. Because of that, I would strongly support deleting them altogether, rather than making available a configuration option.
I've updated Roger's patch for current 3.x tip (and also extended it to remove the reference to tearoff menus from the help file).
|
msg247612 - (view) |
Author: Mark Roseman (markroseman) * |
Date: 2015-07-29 19:42 |
Same changes for 2.7 branch
|
msg247649 - (view) |
Author: Terry J. Reedy (terry.reedy) *  |
Date: 2015-07-30 03:39 |
In your online tutorial, your recommend
root.option_add('*tearOff', FALSE)
Why not here, and be covered everywhere, and in the future?
Or is this needed for each Toplevel?
|
msg247661 - (view) |
Author: Tal Einat (taleinat) *  |
Date: 2015-07-30 11:28 |
+1 for removal.
|
msg247676 - (view) |
Author: Mark Roseman (markroseman) * |
Date: 2015-07-30 14:52 |
Doing it via the option database vs. on each menu would be my preferred approach. The option database is global, not per toplevel, so would cover everything. Only 'downside' is the whole its-not-an-application-its-a-library thing, though in the highly unlikely case someone was actually using idlelib this way, I'd consider setting that option a public service.
There's no harm doing it the other way (and I was being polite since the previous patch was done that way), but it does very marginally increase maintainability difficulties (remembering to keep the option and add it to any future menus).
Either way - please let's just make this change! :-)
|
msg247694 - (view) |
Author: Roundup Robot (python-dev)  |
Date: 2015-07-30 20:45 |
New changeset bead9330438c by Terry Jan Reedy in branch '2.7':
Issue #13884: Idle: Remove tearoff lines from menus. Patch by Roger Serwy.
https://hg.python.org/cpython/rev/bead9330438c
New changeset 6eb4441ed14b by Terry Jan Reedy in branch '3.4':
Issue #13884: Idle: Remove tearoff lines from menus. Patch by Roger Serwy.
https://hg.python.org/cpython/rev/6eb4441ed14b
|
msg247697 - (view) |
Author: Terry J. Reedy (terry.reedy) *  |
Date: 2015-07-30 20:58 |
I decided the accessibility argument does not apply because a) there is only one nested menu, Recent files b) that menu can be very wide and I intend to make it longer, making it a bad candidate for staying on the screen, and c) when torn off, it did not seem accessible from the keyboard.
I decided to keep the patch as is, with tearoff disabled at the point of submenu creation. Setting the option on root is no easier when there are multiple Tk() calls. There may be more than one for normal Idle execution. There are many for testing, and we want the option set for tests.
|
|
Date |
User |
Action |
Args |
2022-04-11 14:57:26 | admin | set | github: 58092 |
2020-06-08 00:29:28 | terry.reedy | unlink | issue13504 dependencies |
2015-07-30 21:11:24 | terry.reedy | set | assignee: terry.reedy |
2015-07-30 20:58:05 | terry.reedy | set | status: open -> closed resolution: fixed messages:
+ msg247697
stage: patch review -> resolved |
2015-07-30 20:45:19 | python-dev | set | nosy:
+ python-dev messages:
+ msg247694
|
2015-07-30 14:52:07 | markroseman | set | messages:
+ msg247676 |
2015-07-30 11:28:26 | taleinat | set | nosy:
+ taleinat messages:
+ msg247661
|
2015-07-30 03:39:41 | terry.reedy | set | messages:
+ msg247649 versions:
+ Python 3.6 |
2015-07-29 19:42:52 | markroseman | set | files:
+ tearoff27.patch
messages:
+ msg247612 |
2015-07-29 19:42:13 | markroseman | set | files:
+ tearoff.patch nosy:
+ markroseman messages:
+ msg247611
|
2014-10-31 19:27:47 | ezio.melotti | set | messages:
+ msg230381 |
2014-10-20 04:59:27 | terry.reedy | link | issue13504 dependencies |
2014-10-03 03:50:10 | terry.reedy | set | versions:
+ Python 2.7, Python 3.5 |
2012-10-18 21:53:23 | ned.deily | set | nosy:
+ ned.deily messages:
+ msg173300
|
2012-10-18 20:12:35 | serhiy.storchaka | set | nosy:
+ serhiy.storchaka messages:
+ msg173293
|
2012-10-18 19:54:46 | Todd.Rovito | set | messages:
+ msg173291 |
2012-10-18 17:04:19 | terry.reedy | set | messages:
+ msg173284 |
2012-10-18 16:57:29 | terry.reedy | set | title: IDLE 2.6.5 Recent Files undocks -> IDLE: Remove tear-off menu feature stage: patch review type: enhancement versions:
+ Python 3.4, - Python 2.6 |
2012-10-18 16:05:45 | Todd.Rovito | set | nosy:
+ Todd.Rovito
|
2012-01-28 21:54:03 | roger.serwy | set | files:
+ issue13884.patch
nosy:
+ roger.serwy messages:
+ msg152192
keywords:
+ patch |
2012-01-27 06:45:39 | terry.reedy | set | messages:
+ msg152067 |
2012-01-26 23:58:20 | ezio.melotti | set | nosy:
+ ezio.melotti, terry.reedy messages:
+ msg152054
|
2012-01-26 23:41:03 | mcgrete | create | |