Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325694 - [call hierarchy] Delete key should remove element from view instead of deleting it
Summary: [call hierarchy] Delete key should remove element from view instead of deleti...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.7 M3   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-19 05:09 EDT by Andrey Loskutov CLA
Modified: 2010-10-26 02:19 EDT (History)
3 users (show)

See Also:


Attachments
Fix (9.59 KB, patch)
2010-09-20 09:06 EDT, Dani Megert CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Loskutov CLA 2010-09-19 05:09:56 EDT
Call hierarchy could be extended in the way that it's behavior matches the "standart" behavior of the search view.

1) "Delete" action deletes the Java element from the *code*. This is VERY inconvenient. Expected: as in the Search view, the *search result* is removed from view. Additionally, "del" keybinding should be used by the "remove from view" context menu.

2) There is no way to open multiple call hierarchies at same time. Expected: as in the Search view, "Pin this view" button should be available at the view toolbar.
Comment 1 Dani Megert CLA 2010-09-20 03:42:50 EDT
(In reply to comment #0)
> Call hierarchy could be extended in the way that it's behavior matches the
> "standart" behavior of the search view.
> 
> 1) "Delete" action deletes the Java element from the *code*. This is VERY
> inconvenient. Expected: as in the Search view, the *search result* is removed
> from view. Additionally, "del" keybinding should be used by the "remove from
> view" context menu.
I agree. Actually we discussed this in the team already.

 
> 2) There is no way to open multiple call hierarchies at same time. Expected: as
> in the Search view, "Pin this view" button should be available at the view
> toolbar.
See bug 304135.
Comment 2 Dani Megert CLA 2010-09-20 09:05:59 EDT
Fixed in HEAD.
Available in builds >= N20100920-2000.
Comment 3 Dani Megert CLA 2010-09-20 09:06:33 EDT
Created attachment 179242 [details]
Fix
Comment 4 Raksha Vasisht CLA 2010-10-26 02:19:22 EDT
Verified for 3.7 M3 with I20101025-1800.