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

Bug 317201

Summary: key assist (Ctrl+Shift+L) does not work
Product: [Eclipse Project] e4 Reporter: Dani Megert <daniel_megert>
Component: UIAssignee: Jesse <jessev>
Status: VERIFIED FIXED QA Contact: Paul Webster <pwebster>
Severity: major    
Priority: P3 CC: bokowski, kitlo, Lars.Vogel, mlists, pwebster, remy.suen, thatnitind
Version: 1.0   
Target Milestone: 4.1 RC1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Work in Progress ... keyAssist v01
none
keyAssist v02
none
keyAssist v03 none

Description Dani Megert CLA 2010-06-17 11:19:42 EDT
http://download.eclipse.org/e4/sdk/drops/I20100616-2127/index.php.

Key Assist (Ctrl+Shift+L) does not work.
Comment 1 Paul Webster CLA 2010-07-16 12:03:03 EDT
Created attachment 174511 [details]
Work in Progress ... keyAssist v01
Comment 2 Remy Suen CLA 2010-12-09 10:33:12 EST
*** Bug 332213 has been marked as a duplicate of this bug. ***
Comment 3 Paul Webster CLA 2011-03-11 08:43:54 EST
Jesse, have a look at the key assist dialog.  You need a couple of things:

1) to support in in e4, there needs to be some way to hook into org.eclipse.e4.ui.bindings.keys.KeyBindingDispatcher.  See org.eclipse.ui.internal.keys.WorkbenchKeyboard in your 3.7 workspace. We can create a KeyAssistDialog in org.eclipse.e4.ui.bindings.  Or enhance org.eclipse.e4.ui.bindings.keys.KeyBindingDispatcher to be able to use a dialog contributed from somewhere else, where that somewhere else is the compatibility layer.

2) you need to be able to open the dialog with the list of active keybindings.

Once that works, then we need:

3) you need to be able to ask the EBindingService (runtime) if there are any conflicts.  That implies we need to be able to store conflicts in one BindingTable.  We also need to be able to report conflicts from 2 binding tables at the same context level (i.e. 2 sibling contexts, see org.eclipse.e4.ui.bindings.internal.ContextSet for a definition of level).

4) if there are conflicts, we need to be able to open the key assist dialog with the short list.

PW
Comment 4 Dani Megert CLA 2011-03-18 03:42:31 EDT
*** Bug 340320 has been marked as a duplicate of this bug. ***
Comment 5 Paul Webster CLA 2011-04-29 15:02:05 EDT
Created attachment 194395 [details]
keyAssist v02
Comment 6 Paul Webster CLA 2011-05-11 15:46:28 EDT
Created attachment 195417 [details]
keyAssist v03

Slight cleanup of patch.

PW
Comment 7 Paul Webster CLA 2011-05-11 15:53:25 EDT
(In reply to comment #6)
> Created attachment 195417 [details]
> keyAssist v03
> 

Released.  Thanx Jesse!

PW
Comment 8 Dani Megert CLA 2011-05-16 07:02:26 EDT
Verified in 4.1 build: I20110515-0800: it only works partially: while I can bring up the dialog now, I cannot go to the preference page from there via Ctrl+Shift+L anymore. The hint is also missing. Filed bug 345916 to track that.
Comment 9 Dani Megert CLA 2011-05-16 09:34:40 EDT
There's also bug 345946.