Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 205331

Summary: [GlobalActions] Move "Uncategorized > Open File..." to "File > Open File..."
Product: [Eclipse Project] Platform Reporter: Madhavi Balusu <madhavi.balusu>
Component: UIAssignee: Paul Webster <pwebster>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: nboldt, pwebster, remy.suen
Version: 3.3Keywords: helpwanted
Target Milestone: 3.4 M3   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
the Window>Preferences>General>Keys still does not show File->File Open when I unchecked "filter uncategorized commands"
none
Using Eclipse 3.3.0, I can't find it either... looks like it's not just unbound, it's unexposed.
none
Patch to add a commands extension. none

Description Madhavi Balusu CLA 2007-10-03 14:11:49 EDT
Build ID: I20070621-1340

Steps To Reproduce:
1. Open Eclipse
2.Look at File->”Open File…” menu, it has no accelerator to it. 
3.Expected: Add Ctrl+O as the accelerator key.


More information:
Plus also make it available in windows->Preferences->General->Keys
Comment 1 Kevin McGuire CLA 2007-10-03 20:35:57 EDT
Cntrl-O is currently used by JDT for Quick-Outline.
File->Open File... isn't very common in Eclipse workflows.  While its true that's the common accelerator, I think people would prefer keeping it bound to Quick-Outline by default.
Comment 2 Paul Webster CLA 2007-10-04 12:28:24 EDT
You can still set a binding for yourself from Window>Preferences>General>Keys

You might have to go to the Advanced button and uncheck "filter uncategorized commands"

PW
Comment 3 Madhavi Balusu CLA 2007-10-04 13:22:20 EDT
Created attachment 79754 [details]
the Window>Preferences>General>Keys still does not show File->File Open when I unchecked "filter uncategorized commands"
Comment 4 Madhavi Balusu CLA 2007-10-04 13:28:01 EDT
It does not matter what the accelerator key is as long you provide an option to add one in Window>Preferences>General>Keys. It's glaringly being shown in File menu, what is the big secret being acheived by not allowing to bind an accelerator in Window>Preferences>General>Keys????!!!Does not make sense!!!

Comment 5 Remy Suen CLA 2007-10-04 13:39:18 EDT
It's not in the 'File' category but is in the 'Uncategorized' category. Type 'open f' in the filter text control below the 'Scheme' combo box and above the table and you'll find it.
Comment 6 Madhavi Balusu CLA 2007-10-04 13:51:39 EDT
Thanks that worked. Thanks for your fast response. I am impressed.
Comment 7 Nick Boldt CLA 2007-10-04 13:51:41 EDT
Created attachment 79756 [details]
Using Eclipse 3.3.0, I can't find it either... looks like it's not just unbound, it's unexposed.
Comment 8 Remy Suen CLA 2007-10-04 13:55:41 EDT
Hi Nick, good catch. You also have to press the 'Advanced' button and then uncheck 'Filter uncategorized commands' in the subsequent dialog. Sorry about that, I have them unfiltered so I didn't include this step in comment #5.
Comment 9 Nick Boldt CLA 2007-10-04 13:57:37 EDT
(In reply to comment #7)
> Created an attachment (id=79756) [details]
> Using Eclipse 3.3.0, I can't find it either... looks like it's not just
> unbound, it's unexposed.

Whoops, my bad...

Advanced > [ ] Filter uncategorized commands

Found it. ;-)

But this brings up a usability question. Why isn't it categorized to match where it lives on the menu? If it's File > Open File... shouldn't it be categorized in the File category? 

Seems counterintuitive to a) not be categorized and b) have it hidden under the Advanced filters by default.

Reopening and renaming to reflect actual problem.
Comment 10 Madhavi Balusu CLA 2007-10-04 13:58:24 EDT
Here are the steps to bind an accelerator to File->Open File... 

1. Open windows->Preferences->General->Keys
2. Make sure "Include unbound commands" is checked.
3. Click advanced and uncheck "filter uncategorized
commands" (actually i unchecked all 3 of the check boxes there, but I don't think you have to).
Type
4. Click 'open f' in the filter text control below the 'Scheme' combo box and above the table and you'll find it.

Hope that helps.


Comment 11 Nick Boldt CLA 2007-10-04 14:04:36 EDT
Madhavi:

Yes, it's clearly doable. 

My point is that it's counterintuitive and sloppy to not be in File.

But it sounds like this will be fixed with bug 186522, so I'll just link them together and hope that patch gets applied sooner rather than later.

Renaming again to reflect the actual problem that's surfaced out of this discusssion.
Comment 12 Madhavi Balusu CLA 2007-10-04 14:21:35 EDT
Nick, you are right!
Comment 13 Paul Webster CLA 2007-10-04 14:28:56 EDT
:-)

OK, now that I've read to the end ... this is a platform action that should be represented by a command (and it's not).

Comment 14 Paul Webster CLA 2007-10-04 14:29:33 EDT
Open File needs a definitionId and matching command
PW
Comment 15 Remy Suen CLA 2007-10-04 15:23:29 EDT
Created attachment 79766 [details]
Patch to add a commands extension.

It seems the action set already has a definitionId set so I just took that one.
Comment 16 Paul Webster CLA 2007-10-22 13:12:30 EDT
Released to HEAD >20071022
PW
Comment 17 Paul Webster CLA 2007-10-23 14:41:19 EDT
in HEAD
PW
Comment 18 Paul Webster CLA 2007-10-30 09:59:51 EDT
In I20071030-0010
PW