HomePhabricator
Diffusion MITK d0195cd80a6b

Remove StdMultiWidget dependency

Description

Remove StdMultiWidget dependency

Summary:
The basic idea is to provide a QmitkAbstractMultiWidget and a QmitkAbstractMultiWidgetEditor which is subclassed by the corresponding multi widget and editor classes:

  • QmitkStdMultiWidget / QmitkStdMultiWidgetEditor
  • QmitkMxNMultiWidget / QmitkMultiWidgetEditor

Basic functionally, getter, setter etc. is moved to the abstract multi widget (editor) classes to provide common behavior.

Additionally a layout manager has been introduced to provide generalized layouts that apply for different multi widget (editors). In order to to this, the QmitkRenderWindow and the QmitkRenderWindowMenu had to be refactored (and cleaned simplified).

The multi widgets do not hold a QmitkRenderWindow directly but a QmitkRenderWindowWidget, which provides functions to change the decorations and colors.

Since a lot of classes still access render windows via ("axial", "sagittal", "coronal", "3d"), a workaround was added to both multi widgets. This will be fixed in T25595.

Test Plan:

  • manually tested with Windows
  • merging this will include user tests on different systems (mainly GUI changes)

Reviewers: MITK Reviewer Group I, O1 MITK Core, floca

Reviewed By: MITK Reviewer Group I, O1 MITK Core, floca

Subscribers: kislinsk, floca, kalali

Tags: MITK

Maniphest Tasks: T24215

Differential Revision: https://phabricator.mitk.org/D212

Details

Provenance
kalaliAuthored on Oct 31 2019, 12:46 PM
kalaliPushed on Nov 5 2019, 11:35 AM
Reviewer
MITK Reviewer Group I
Differential Revision
Restricted Differential Revision
Parents
rMITKffbd515105fd: Merge branch 'T24215-Remove-stdmultiwidget-dependency'
rMITK80bc0f732072: Merge branch 'T26779-Convert-to-UTF-8'
Branches
Unknown
Tags
Unknown

Event Timeline

Merged Changes

This commit merges a very large number of changes. Only the first 50 are shown.