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

Bug 458581

Summary: Remove keybindings for quickfixes
Product: [ECD] Orion Reporter: Michael Rennie <Michael_Rennie>
Component: JS ToolsAssignee: Michael Rennie <Michael_Rennie>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: emoffatt, john.arthorne
Version: 8.0   
Target Milestone: 8.0   
Hardware: All   
OS: All   
Whiteboard:

Description Michael Rennie CLA 2015-01-27 15:53:54 EST 
If a keybinding is set on a quick fix as soon as you open a source file (of any kind), the Keyboard Shortcuts drop down fills with the bindings for all fixes (in the Globals section). This is a pretty bad presentation.

Since we haven't completely sorted out how (or if) bindings will work for fixes, we should remove any plugin-spcific bindings to avoid making a mess of the bindings drop down for commands that won't work outside of the tootip context.
Comment 2 John Arthorne CLA 2015-01-28 21:33:40 EST 
Does this mean there is no easy way to apply a quick fix from keyboard? I would expect to be able to do F2, then Down Arrow to select and Enter to apply.
Comment 3 Michael Rennie CLA 2015-01-29 09:21:25 EST 
(In reply to John Arthorne from comment #2)
> Does this mean there is no easy way to apply a quick fix from keyboard? I
> would expect to be able to do F2, then Down Arrow to select and Enter to
> apply.

That still works (keyboard over to the annotation, F2, arrow key, enter -> applies fix). But the slew of individual bindings for each quick fix (which all just happened to be Ctrl+Shift+E) have been removed since they were all applying to the 'Global' context (which would not work anyway).

Eric is working on more of the binding support in bug 451981.