This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 415473 - Persisting bindings forever results in bad/conflicting bindings that cannot be removed
Summary: Persisting bindings forever results in bad/conflicting bindings that cannot b...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.3   Edit
Hardware: PC Windows 7
: P3 major (vote)
Target Milestone: 4.3.1   Edit
Assignee: Paul Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 406003
Blocks:
  Show dependency tree
 
Reported: 2013-08-20 07:56 EDT by Paul Webster CLA
Modified: 2013-08-28 10:07 EDT (History)
9 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Webster CLA 2013-08-20 07:56:47 EDT
Backport to 4.3.1

+++ This bug was initially created as a clone of Bug #406003 +++

Every key binding that is loaded from the bindings extension is persisted in the workbench xmi.  If the plug-in providing that extension is later removed, the keybinding stays.  Worse is that the keys pref page doesn't recognize the conflict.  The only workaround is to delete the xmi file or start a new workspace.

1) Create a new plug-in project > Create an RCP > Use the Mail RCP Template
2) Launch a normal Eclipse application launch (runtime workbench)
3) Note that CTRL+X will close Eclipse instead of copy
4) Note that CTRL+3 has a conflict (Quick Access and Open Message)
5) Shut down the runtime workbench
6) Close or delete the Mail RCP project
7) Launch the runtime workbench again
Result:
CTRL+X still shuts down Eclipse!
CTRL+3 still has a conflict
The preference page doesn't list any conflicts for those keybindings
Comment 1 Paul Webster CLA 2013-08-20 07:58:46 EDT
backport: https://git.eclipse.org/r/15649
Comment 2 Paul Webster CLA 2013-08-20 07:59:09 EDT
Paul, could you please review this?
Comment 4 Paul Elder CLA 2013-08-26 15:06:10 EDT
Verified in build 4.3.0.M20130821-0800
Comment 5 Paul Elder CLA 2013-08-28 10:06:51 EDT
Per comment 4, state should be Verified
Comment 6 Paul Elder CLA 2013-08-28 10:07:02 EDT
Verified per comment 4.