Bug 63641 - Menu cleanup in JDT UI
Summary: Menu cleanup in JDT UI
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 RC1   Edit
Assignee: Dirk Baeumer CLA
QA Contact:
URL:
Whiteboard:
Keywords: polish
Depends on:
Blocks:
 
Reported: 2004-05-24 05:02 EDT by Erich Gamma CLA
Modified: 2004-06-17 19:32 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Erich Gamma CLA 2004-05-24 05:02:12 EDT 
* Navigate 
 - Open type Hierarchy (F4) should become dynamic based on the 
     preference setting
	1) Open type hierarchy (F4) opens type hierarchy perspective
	2) Show type hierarchy (F4) shows the type heirarchy view
           in the current perspective	
 -  Open Call Heirarchy  TO Show Call Heirarchy
 -  Show Outline (Ctrl+O) TO Quick Outline
 -  Open type Hierarchy (Ctrl+t) TO Quick Type Hierarchy

Package Explorer
 -  Cut  only show cut when the show members preference is enabled.
             cut is only ever enabled on source elements.
             Fallback is to only add cut when the action is enabled.

Refactor
 -  move introduce paramater to below change method signature
Comment 1 Dirk Baeumer CLA 2004-05-25 09:39:48 EDT 
Here is what I did:

- kept the Open since it was to confusing with the Show in
- renamed Show Outline/Type Hierarchy to Quick ...
- removed Cut from the context menu if disabled. The goupd doesn't know
  anything about the package explorer and I don't want to introduce the
  dependency (class is API) so I removed it if the action is disabled.
- didn't reorder introduce parameter since it is more comparable to 
  extract local, constant. You select an expression and convert it into
  another element.
Comment 2 Dirk Baeumer CLA 2004-05-25 09:43:24 EDT 
Released...
Comment 3 Thomas M??der CLA 2004-06-11 12:03:45 EDT 
start verifying.
Comment 4 Thomas M??der CLA 2004-06-11 12:10:24 EDT 
Problem: Cut is not enabled for types. Verified the other changes.
Comment 5 Dirk Baeumer CLA 2004-06-17 19:32:21 EDT 
Cut wasn't disabled in the past for type (checked in M9). So the fix is OK.