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 43089

Summary: Fix contextual menu for Java package node
Product: java Reporter: jrojcek <jrojcek>
Component: ProjectAssignee: Petr Hrebejk <phrebejk>
Status: RESOLVED FIXED    
Severity: blocker Keywords: UI
Priority: P4    
Version: 4.x   
Hardware: All   
OS: All   
URL: http://projects.netbeans.org/nonav/buildsys/j2se-project-ui-spec.html#Contextual_Menu_A_Java_Package_N
Issue Type: DEFECT Exception Reporter:
Bug Depends on:    
Bug Blocks: 41835    

Description jrojcek 2004-05-11 15:12:12 UTC
It is not exactly according to the spec.
Comment 1 Petr Hrebejk 2004-05-31 16:42:08 UTC
What do you mean? The position of refresh and it's separation?


Or add files (this is probably WONTFIX, isn't it?)
Comment 2 jrojcek 2004-05-31 22:02:40 UTC
Mostly the position of New and Refresh.

Yes, "Add Existing" is wontfix (for now ;-).
Comment 3 Petr Hrebejk 2004-06-01 09:08:35 UTC
New is fixed now. I can move the refresh to the right position. But it
will still have one extra separator. (Not in the spec). This is
because the refresh is in fact filesystem action (contains VCS
operations when working on VCS folder) and is presented by the evil
JInline menu. We wont fix this into D.
Comment 4 jrojcek 2004-06-01 09:17:30 UTC
Okay, so lets keep this open so that we don't forget about the evil JInline menu in this 
contextual menu.
Comment 5 Petr Hrebejk 2004-06-01 10:22:30 UTC
I've changed the ordering according to spec. Everywhere not only in
the JavaPackage node but also in physical view and favourites tab. If
you want the filesystem action not to have separators in some cases
please file another bug against loaders or filesystems or what ever. 

If you want not to have CVS actions on Javapackages reopen this bug.
Comment 6 Jesse Glick 2004-06-01 14:46:40 UTC
I think we have to remove FileSystemAction from Java package nodes
unless and until the VCS team provides an API to make it not
recursively traverse folders during the VCS operation. Better to omit
VCS actions than have them not work as expected.
Comment 7 Petr Hrebejk 2004-06-01 14:58:12 UTC
OK. Agree. Reopening.
Comment 8 Petr Hrebejk 2004-09-07 09:12:06 UTC
Action removed

Checking in
java/project/src/org/netbeans/spi/java/project/support/ui/PackageViewChildren.java;
/cvs/java/project/src/org/netbeans/spi/java/project/support/ui/PackageViewChildren.java,v
 <--  PackageViewChildren.javanew revision: 1.38; previous revision: 1.37
done
Processing log script arguments...
More commits to come...
Checking in ide/test/qa-functional/src/validation/IDEValidation.java;
/cvs/ide/test/qa-functional/src/validation/IDEValidation.java,v  <-- 
IDEValidation.java
new revision: 1.74; previous revision: 1.73
done