Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 337299

Summary: context-menu in package explorer should be context sensitiv
Product: z_Archived Reporter: Sebastian Proksch <sebastian>
Component: RecommendersAssignee: Marcel Bruch <marcel.bruch>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3    
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:

Description Sebastian Proksch CLA 2011-02-16 06:09:37 EST
I think it would be a nice enhancement, if the contextmenu for the package explorer would be context-sensitiv with respect to the presence/absence of the recommenders nature.

At the moment the only entry is "add/remove nature", the text could switch between "add nature" and "remove nature", according to the state of the project.

A later use for this could be to hide/disable options from the context menu, that only make sense for nature-enabled projects. As another benefit the user is able to discover the nature status just by rigth-clicking the project, instead of looking into the projects properties... (compare the "Team" entry for SVN enabled projects)

Surely this is common sense, I just wrote it down for the sake of completeness :)
Comment 1 Marcel Bruch CLA 2011-02-18 13:25:41 EST
Hi Sebastian,

you're right. This should be changed. We will schedule this for next release.

Thanks,
Marcel
Comment 2 Marcel Bruch CLA 2011-02-20 04:49:46 EST
Proposed a fix in master. Action shows one of three different labels now:

if all selected projects possess recommenders nature:
 "Remove Recommenders Nature"

if no selected project possesses recommenders nature:
 "Add Recommenders Nature"

if mixture of projects that possess and don't possess recommenders nature are selected:
 "Flip Recommenders Nature"

Does this meet your expectation? Are the labels clear? Other suggestions? :D
Please set to verified if you are good with this proposal.
Comment 3 Marcel Bruch CLA 2012-02-02 01:41:42 EST
Closing this issue. Please give latest dev build (v0.5) a try:

http://download.eclipse.org/recommenders/updates/dev/e37/
http://download.eclipse.org/recommenders/updates/dev/e42/
Comment 4 Marcel Bruch CLA 2012-02-02 01:41:55 EST
Closing this issue. Please give latest dev build (v0.5) a try:

http://download.eclipse.org/recommenders/updates/dev/e37/
http://download.eclipse.org/recommenders/updates/dev/e42/