Comment 1 Darin Wright 2001-12-31 11:16:59 EST

Agreed. There should be an action package. 

Where possible, actions should be contributed via plug-in XML rather than 
hardcoded by a view. Also, action updating (enabling) should be handled by the 
action rather than the view where possible. This reduces the cohesion between 
actions and views, and makes the actions more portable/reuseable.

Not sure if a bug has been filed against the workbench, but it is crucial 
that "view action delegates" get a "dispose" callback such that they can remove 
themselves as event listeners, etc. This is one roadblock I have found in using 
action contributions effectively.

Comment 2 Darin Swanson 2001-12-31 12:51:09 EST

A dispose callback would work for bug 7010 as well...the reason I wanted to 
know when the last editor was closed was to remove the action delegate as a 
listener.

Comment 3 Nick Edgar 2002-01-03 10:26:37 EST

Filed bug 7235 for the dispose problem.

Comment 4 Darin Swanson 2002-02-09 17:14:11 EST

The current workaround for not having a dispose call on an IViewActionDelegate 
is to add the action as a IPartListener. When the part closes, this is treated 
as a dispose.

Comment 5 Darin Wright 2002-05-14 10:39:47 EDT

Deferred until 7235 is fixed.

Comment 6 Darin Swanson 2002-06-26 11:16:52 EDT

Resolving to later.

Comment 7 Darin Swanson 2003-04-01 17:01:42 EST

We are using the IActionDelegate2 interface and the #dispose method this 
provides.

Closing.

Comment 8 Denis Roy 2009-08-30 02:20:50 EDT

As of now 'LATER' and 'REMIND' resolutions are no longer supported.
Please reopen this bug if it is still valid for you.