This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 68649 - Keymap|Restore does not work as expected
Summary: Keymap|Restore does not work as expected
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: Options (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: issues@editor
URL:
Keywords: RANDOM
: 90768 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-11-14 13:32 UTC by Max Sauer
Modified: 2008-08-13 09:50 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Max Sauer 2005-11-14 13:32:44 UTC
The restore button inside Keymap section of new OD does not work as expected:
- it does not restore overriden shortcuts (this is true only for some elements,
ie. 'Cut' action)
- it does not remove newly defined ones

Steps to reproduce
-------------------
1) Run the IDE with fresh userdir
2) Open a project
3) Open Tools|Options
4) Goto Keymaps
5) Add some new shortcuts, remove some
6) Confirm OD
7) Invoke Tools|Options|Keymap again
8) Press Restore
---> Newly added shortcuts won't be removed
---> Some of the overridden ones are not restored

[051114 ~11:30, JDK 1.5.0_05, solaris 10/sparc]
Comment 1 Jan Jancura 2005-11-30 21:58:24 UTC
reproducible, there are some implementation problems
Comment 2 Jan Jancura 2006-01-03 09:02:19 UTC
Too late to fix in nb5.0.
Comment 3 Vitezslav Stejskal 2007-08-21 15:06:39 UTC
Partially fixed. Unfortunately global shortcuts (those under /Keymaps and /Shortcuts) still do not support reverting.

Checking in core/options/keymap/src/org/netbeans/modules/options/keymap/KeymapModel.java;
/cvs/core/options/keymap/src/org/netbeans/modules/options/keymap/KeymapModel.java,v  <--  KeymapModel.java
new revision: 1.4; previous revision: 1.3
done
Checking in core/options/keymap/src/org/netbeans/modules/options/keymap/LayersBridge.java;
/cvs/core/options/keymap/src/org/netbeans/modules/options/keymap/LayersBridge.java,v  <--  LayersBridge.java
new revision: 1.4; previous revision: 1.3
done
Checking in core/options/keymap/src/org/netbeans/core/options/keymap/spi/KeymapManager.java;
/cvs/core/options/keymap/src/org/netbeans/core/options/keymap/spi/KeymapManager.java,v  <--  KeymapManager.java
new revision: 1.3; previous revision: 1.2
done
Checking in editor/options/src/org/netbeans/modules/options/editor/keymap/EditorBridge.java;
/cvs/editor/options/src/org/netbeans/modules/options/editor/keymap/EditorBridge.java,v  <--  EditorBridge.java
new revision: 1.5; previous revision: 1.4
done
Comment 4 tboerkel 2007-09-18 08:16:44 UTC
Issue #90768 is a duplicate of this one, right?
Comment 5 Vitezslav Stejskal 2007-09-18 11:45:50 UTC
*** Issue 90768 has been marked as a duplicate of this issue. ***
Comment 6 David Strupl 2007-09-18 12:20:54 UTC
Don't miss the attachment I made to #90768 but I guess you know better ;-)
Comment 7 Jan Jancura 2008-02-20 12:23:36 UTC
Works for me. I have tested this issue with Copy, Cut, Clear Document Bookmarks, and given usecase is not reproducible
for me on 080213. So it looks like this issue is already fixed.
Comment 8 Vitezslav Stejskal 2008-02-21 15:44:54 UTC
Well, I'm not sure how you tested it, but it doesn't seem to be working in my dev build. I tried with Edit -> Cut
action, removed CUT shortcut and added ctrl+alt+shift+H. Following the steps I was not able to restore the shortcuts for
that action. IIRC the LayerBridge does not implement restore, because it's not easy to do so for global shortcuts
folders. Yarda once told me that there is a trick in our system filesystem that allows deleting user changes and
restoring the state from bundled module layers. We could possibly use it, but I'm not sure about details.
Comment 9 Vitezslav Stejskal 2008-08-13 09:50:56 UTC
Hmm, it looks like this is working now in current dev builds. Perhaps fixed by some other fixes in the keymaps panel.