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

Bug 358963

Summary: Goto line key binding should not use prompt
Product: [ECD] Orion Reporter: Silenio Quarti <Silenio_Quarti>
Component: EditorAssignee: Susan McCourt <susan>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: mamacdon
Version: 0.2   
Target Milestone: 0.4 M1   
Hardware: PC   
OS: All   
Whiteboard:
Bug Depends on: 342739    
Bug Blocks:    

Description Silenio Quarti CLA 2011-09-26 15:05:40 EDT
The prompt dialog opened by goto line is very annoying.  We should use something similar to the editor search field or the incremental find.
Comment 1 Susan McCourt CLA 2011-11-11 13:55:21 EST
I'm working on this, but not done yet.
In the lower level TextActions, we don't have knowledge of the outer page, so window prompt is going to have to be there.

But up in editorCommands, we can use the command parameter stuff to get the parameter with a slideout, much like find and replace.  As a first step I added  "goto line" to the editor toolbar.  When invoked from the command, the right thing happens (no prompt, just a little slide out). 

What I still need to do (next week) is to override the editor keybinding to do the command/parameter invocation rather than the underlying TextActions implementation.  For this to work, there is some cleanup I need to do in the command framework to gather the right parameters (such as command dom ids) that are needed when the command executes from a keybinding.
Comment 2 Susan McCourt CLA 2011-11-15 12:36:46 EST
fixed.