Bug 110232 - [KeyBindings] preference page: adding keybinding removes keybinding from more specific context
Summary: [KeyBindings] preference page: adding keybinding removes keybinding from more...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: ali sheraz CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 176235
Blocks:
  Show dependency tree
 
Reported: 2005-09-21 14:58 EDT by Randy Hudson CLA
Modified: 2007-05-01 08:15 EDT (History)
1 user (show)

See Also:

Attachments
Binding Manager v01 (3.27 KB, patch)
2007-04-24 21:16 EDT, Paul Webster CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Randy Hudson CLA 2005-09-21 14:58:26 EDT 
I want to define CTRL+RIGHT to be "Navigate/Go Into" in the windows scope.
When I do this, "Text/Next Word" in the "Editing Text" scope gets removed. This
shouldn't happen, since that scope is more specific and overrides the
keybinding, which is what I would want.

The workaround is to go back to the "Next Word" command and re-add its original
binding.
Comment 1 Michael Van Meekeren CLA 2006-04-21 13:14:31 EDT 
Moving Dougs bugs
Comment 2 Paul Webster CLA 2006-09-28 15:15:29 EDT 
Is this still a problem in 3.3?

PW
Comment 3 Randy Hudson CLA 2006-09-28 16:57:55 EDT 
yes
Comment 4 Paul Webster CLA 2007-04-24 21:16:37 EDT 
Created attachment 64811 [details]
Binding Manager v01

The code that was supposed to build a list of all keybindings would remove any SYSTEM keybindings that it found first.

PW
Comment 5 Randy Hudson CLA 2007-04-25 10:01:55 EDT 
I recently re-encountered this bug and nearly opened a dupe of myself. The reason this is so annoying is that the conflict is often assumed but doesn't really exist. The example mentioned here already shows thi. "Go Into" is never enabled at the same time as "Next Word", yet the preference page assumes a conflict based on context inheritance, and re-maps the existing binding.
Comment 6 Paul Webster CLA 2007-04-30 14:12:27 EDT 
Already in I build.
PW
Comment 7 Randy Hudson CLA 2007-04-30 15:24:42 EDT 
Does the solution also handle the scenario where the two contexts are orthogonal?
Comment 8 Paul Webster CLA 2007-04-30 18:43:50 EDT 
Now adding a keybinding will only remove the system keybinding with the same command in the same scope.

PW
Comment 9 Paul Webster CLA 2007-05-01 08:15:58 EDT 
In I20070501-0010
PW