| Summary: | [Commands] request: Provide close view command and binding | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | David W Campbell <dcampbell> | ||||||
| Component: | UI | Assignee: | Paul Webster <pwebster> | ||||||
| Status: | VERIFIED FIXED | QA Contact: | |||||||
| Severity: | enhancement | ||||||||
| Priority: | P3 | CC: | eclipse, john.arthorne, markus.kell.r, tlroche | ||||||
| Version: | 3.0.1 | Keywords: | helpwanted | ||||||
| Target Milestone: | 3.3 M4 | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
|
Description
David W Campbell
There is no command for this yet, so one would need to be added. *** Bug 81466 has been marked as a duplicate of this bug. *** *** Bug 94722 has been marked as a duplicate of this bug. *** Moving Dougs bugs There are currently no plans to work on this feature. PW Created attachment 51093 [details]
Patch implementing close part action
You said helpwanted, so here you go. I wrote this one day after despairing over typing "Alt+-, c" to close a view for the nth time - this is my most frequently used action that is currently not key-bindable.
This patch adds a new "close part" command that will close the active part, either an editor or view. I then manually switch the Ctrl+W key binding to this command instead of the "close editor" command. I took this approach rather than changing the current CloseEditorAction, because I'm sure some people would not want this change in behaviour (if they hit Ctrl+W when the Navigator is in focus, they might really want to close the current editor rather than the Navigator).
excellent. PW Ooops, missed M3, I'll put it in first thing next week. PW I've still got this in my workspace, so ping me when you're ready for a patch. There have been conflicting changes since the original patch so you would have difficultly applying as is. Created attachment 53478 [details]
close part patch v02
John, I've converted your patch to a command+handler. Could you have a quick look?
You still bind to it in the keys preference page.
PW
But... your patch is too simple! No action factory, no action builder... Can the new command story really be this simple? How am I supposed to impress people with my knowledge of the ten arcane steps needed to add an action to the workbench? +1 Although I think it should be moved to the Window menu (next to Close Perspective) rather than the File menu... I've moved the command into the Window Category from File. I haven't actually placed the command in a menu, and any user that wants it will have to bind the key of their choice to it. Released in HEAD >20061109 PW fixed PW in I20061212-0010 PW |