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 223132

Summary: File template manager: contextmenu items have no visible shortcuts for Cut, Copy, Del, Rename
Product: platform Reporter: markiewb
Component: ActionsAssignee: Jaroslav Tulach <jtulach>
Status: RESOLVED INVALID    
Severity: normal CC: mentlicher, psomol
Priority: P3    
Version: 7.3   
Hardware: PC   
OS: Windows 7   
Issue Type: DEFECT Exception Reporter:
Bug Depends on:    
Bug Blocks: 220835    
Attachments: Shows the context menu

Description markiewb 2012-12-02 14:11:37 UTC
[ BUILD # : 201211290002 ]
[ JDK VERSION : 1.7.0_09 ]

File template manager: 
ACTUAL:
contextmenu items have no visible shortcuts for Cut, Copy, Del, Rename
See screenshot.

EXPECTED: visible and working shortcuts

FYI: 
* Cut action does not work currently - http://netbeans.org/bugzilla/
show_bug.cgi?id=223131
* Copy, Del actions work via shortcuts, but the shortcuts are not shown in 
the context menu
* Rename action has no shortcut yet (except mnemonic R in &Rename) -> 
should be F2


----
Product Version: NetBeans IDE Dev (Build 201211290002 )
Java: 1.7.0_10-ea; Java HotSpot(TM) 64-Bit Server VM 23.6-b04
Runtime: Java(TM) SE Runtime Environment 1.7.0_10-ea-b17
System: Windows 7 version 6.1 running on amd64; Cp1252; de_DE (nb)
Comment 1 markiewb 2012-12-02 14:11:59 UTC
Created attachment 128712 [details]
Shows the context menu
Comment 2 Martin Entlicher 2012-12-07 15:09:46 UTC
Shortcut for Rename is defined by changeset:   242404:207396553735
http://hg.netbeans.org/core-main/rev/207396553735

We use standard actions for Cut/Copy/Delete.
I have no idea why the shortcuts are not visible.

I've found out, that the action, which comes from the standard org.openide.actions.CutAction:
org.openide.util.actions.CallbackSystemAction$DelegateAction@d36859[delegate=org.openide.actions.CutAction@115744b]
does not define the Action.ACCELERATOR_KEY

Moving to actions for investigation...
Comment 3 Martin Entlicher 2012-12-07 15:12:28 UTC
Paste has the accelerator, but Cut, Copy and Delete does not. Any idea what can be wrong?
Comment 4 Quality Engineering 2012-12-09 03:09:45 UTC
Integrated into 'main-golden', will be available in build *201212090001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/207396553735
User: mentlicher@netbeans.org
Log: #223132: F2 shortcut defined for Rename action.
Comment 5 Jaroslav Tulach 2012-12-10 12:46:15 UTC
> EXPECTED: visible shortcuts

What makes you believe shortcuts should be visible? Reference to some UI spec is enough. 

However I have to admit I always lived in an impression that shortcuts (like F2 for rename) should not be displayed in the popup menu. Marking your report invalid for now.

However I admit there should be a consistency. Once we know whether shortcuts (like F2 for rename) should be or should not be displayed all violations against the expected behavior will be P2 bugs.
Comment 6 Petr Somol 2012-12-10 12:58:50 UTC
A discussion is now going on regarding menu system, mnemonics, etc. within the NB UI Steering Committee. I am adding this to the committee's attention by adding a dependence to the Mnemonics issue, and reference to this issue to http://wiki.netbeans.org/UIReviewsAreas
Comment 7 markiewb 2012-12-10 19:22:23 UTC
(In reply to comment #5)
> > EXPECTED: visible shortcuts
> 
> What makes you believe shortcuts should be visible? Reference to some UI spec
> is enough. 
> 
> However I have to admit I always lived in an impression that shortcuts (like F2
> for rename) should not be displayed in the popup menu. Marking your report
> invalid for now.
> 
> However I admit there should be a consistency. Once we know whether shortcuts
> (like F2 for rename) should be or should not be displayed all violations
> against the expected behavior will be P2 bugs.

It is OK for me, next time i name it as inconsistency. I wondered why and opened this issue.