- TreeViewerEditorBase: Wrap the refresh call into a asyncExec method to avoid invalid tread access when a model is modified which is open in a TreeViewer editor. 

- Introduce a separate base class for composing IModelElements. It manages (among other things) the instantiation of annotations.
- Let HierarchicElementCompositorBase inherit from the newly created base class for IModelElements.

refs 2228

refs 2410

Added a comment in ObjectAwareServiceBase#getRegisteredHandlers stating that this method should be changed if the behavior (from AnntotationValueService) is reused in the future.
refs 2270

refs 2271

refs 2203

refs 2285

- removed all redundant classes AbstractDecoratedComplexCellWithTextStyledTextActionHandlerDatabindingEditingSupport, AbstractComplexCellWithTextStyledTextActionHandlerDatabindingEditingSupport, AbstractComplexCellDatabindingEditingSupport
- changed DataStateValueEditing Support so that complex cells are still supported
refs 2267

refs 2235

refs 2352

- If the creation of new annotations should be wrapped into a command, but the annotation already exists, avoid the usage of a command and just directly return the (already existing) annotation.
- This avoids superfluous model modifications (i.e., the project marked dirty and requires saving)
refs 2369

- Use constants in IPageLayout instead of hard-coded String constants to configure views for StoragePerspective.
- Note: ResourceNavigator view is deprecated since Eclipse 3.5! (warning suppressed for now)

- Since partClosed() is triggered when any IWorkBenchPart is closed, the current implementation erroneously removes the EditorActivationListener even if any other view but the "Model Elements" view is closed.
- Hence, use a different approach and in partActivated() check, if the LibraryView's TreeViewer control is disposed

- Avoid java.lang.IllegalStateException because EditorActivationListener tries to update the input of LibraryView's TreeViewer widget after it has been closed (disposed)

- Switch to non-deprecated variant of KernelModelElementUtils.getParentElement()

refs 2264

refs 1948

refs 1626

refs 2264

- When inserting copies of objects as a result of a copy&paste operation, the inserted model elements is shifted by an offset. This gives visual feedback to the user that the paste operation has actually been performed.
- Notes:
 + Some model types are not handled yet (see ticket)
 + Although maybe an obvious approach, this feature cannot be implemented as a method in HierarchicElementCompositorBase:
   o It is not the base class of all affected compositors
   o HierarchicElementCompositorBase resides in org.fortiss.tooling.base, where as the required methods for detailing with layout information are in org.fortiss.tooling.base.ui
refs 2344

refs 1948

refs 2229

- doCompositorBasedDelete(): Sort list of deletion targets such that object whose eContainer() is not contained in 'targets' are deleted first (e.g., free connectors in hierarchical element models).
- This is required to avoid an NPE when the GUI is refreshed while the list of object to be deleted is processed.

refs 2231

refs 2262