Summary: | [KeyBinding] get best active binding is not working | ||
---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Patrik Suzzi <psuzzi> |
Component: | UI | Assignee: | Platform-UI-Inbox <Platform-UI-Inbox> |
Status: | REOPENED --- | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | bsd, Lars.Vogel, mistria |
Version: | 4.6 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Windows 10 | ||
Whiteboard: | stalebug |
Description
Patrik Suzzi
2016-04-14 10:15:27 EDT
Brian, hope you don't mind if I ask feedback to you As a caller, I expect IBindingService#getBestActiveBindingFor() returns the best binding by searching in all the contexts of the application. With the current behavior the method is searching for the binding only in the active context. This is unclear, as is often unclear which is the active context. My plan is to simply change the implementation of getBestActiveBindingFor() to see the logic that searches in all the contexts, that I think should be replaced as you can see in the current method: ParameterizedCommand cmd = eCommandService.createCommand(MY_COMMAND_ID, null); ContextSet contextSet = manager.createContextSet(Arrays.asList(contextService.getDefinedContexts())); Binding binding = manager.getBestSequenceFor(contextSet, cmd); triggerSequence = binding.getTriggerSequence(); If we search through all contexts then it's not returning the active binding! I'm unable to reproduce this issue on Windows and Linux-GTK. Verified under Windows 10 and Ubuntu 14.04 with the patch at https://git.eclipse.org/r/70706 This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug. If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. -- The automated Eclipse Genie. Still relevant in parts of Eclipse Platform (eg Quick Access) |