| Summary: | [launching] Open Properties dialog by clicking "Run As..." with Ctrl key | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] JDT | Reporter: | Eugene Kuleshov <ekuleshov> | ||||||||
| Component: | Debug | Assignee: | Darin Wright <darin.eclipse> | ||||||||
| Status: | VERIFIED FIXED | QA Contact: | |||||||||
| Severity: | enhancement | ||||||||||
| Priority: | P3 | CC: | andy, jimisola, markus.kell.r, Michael_Rennie | ||||||||
| Version: | 3.1 | ||||||||||
| Target Milestone: | 3.4 M2 | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Eugene Kuleshov
Would require API change/enhancement to ILaunchShortcut. The shortcut is passed the selection to launch and determines which existing launch config to re-use if any. We'd need to pass the corresponding Event along with the request to run, so the shortcut could decide what to do with the modifier. Or, we'd have to ask the short cut to provide us with the associated launch config (if there is one). Deferred. API freeze is upon us. Please consider this as an usability enhancement for 3.3 Created attachment 49870 [details]
proposed fix
This is a proposed fix, containing a new interface to extend for shortcuts called ILaunchShortcut2, which provides two new methods getAssociatedLaunchConfig(ISelection) and getAssociatedLaunchConfig(IEditorPart).
One of the main issues with this fix is consistency. As we do not provide all of the launch shortcuts, Ctrl clicking will work for some but not others
Created attachment 55079 [details]
API patch
This patch improves over the other, by promoting two new API classes for clients to implement: AbstractLaunchShortcut in debug.ui and AbstractJavaLaunchShortcut in jdt.debug.ui.
Created attachment 55080 [details]
small update
same patch as before but with a small update to LaunchShortcutExtension
resolving as later, no plans for this for 3.3 Re-open to consider in 3.4 using the 'smart' framework from context launching, Ctrl+Clicking is supported for launch shortcuts without any additional API. This is how it works: 1. if the Run As item is a config, the dialog is opened on it 2. if the Run As item is a shortcut the launch dialog is opened on what would be contextually launched; in the event there is nothing to be launched or multiple items, the launch dialog is opened to the associated configuration type being selected. Failing those options the launch dialog is opened with no preselection made. 3. if for any reason an error occurs in trying to open the launch dialog or compute what to open on, execution is delegated to the actions' run method fixed in HEAD please verify Darin W Verified. *** Bug 53567 has been marked as a duplicate of this bug. *** |