Bug 334116

Description Nicolas Bros 2011-01-12 10:04:48 EST

In many cases (Query view, FacetSet view, Customization view, EPackage view, Discoverer view, etc.) we need a  view with the following features:
- an Eclipse view containing a tree viewer (with columns) displaying a provided set of elements
- the columns have to be definable by an extension/configuration of the view
- the columns have to be sortable
- the column sizes have to be saved between two uses of the view (including after a restart of Eclipse)
- the view registers an extensible popup menu
- the view has to implement a "delayed-refresh" (only refresh after a delay after the last refresh request, in order to avoid multiple useless consecutive refreshes)
- the extension/configuration of the view can provide a method to define what happens when an element in the view is double-clicked.

Class AbstractCatalogView was created to factor out common functionality and duplicated code between all the "catalog" views in MoDisco (for queries, facets, customizations).

With Bug 333843, I fixed the problem that AbstractCatalogView saved column widths for all subclasses, refactored AbstractCatalogView, extracted a new PersistableColumn class to factor out functionality needed for the new discoverers UI, and created wrappers for JFace columns that work for both tables and trees, so as to be able to handle both in the same class instead of duplicating the code for each.

In parallel, AbstractCatalogView was copied into EMF Facet, and renamed to AbstractTreeView.

Now, in order to reduce maintenance cost by eliminating redundancy, we would like to merge these two forked classes back into one (which will live inside EMF Facet).