Page MenuHomePhabricator

Rework mitkMouseModeSwitcher
Closed, ResolvedPublic

Description

  • the 'mitkMouseModeSwitcher' does not only change the mouse mode but also the interaction scheme and initializes an mitkDisplayInteractor and registers it as 'InteractionEventObserver'-service.
    • too much responsibilities for the single class
    • service registration can happen in the broadcast (service) class itself
  • documentation does not fit the implementation

Revisions and Commits

rMITK MITK
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

kalali triaged this task as Normal priority.Mar 7 2018, 5:17 PM
kalali created this task.
kalali removed kalali as the assignee of this task.Nov 5 2018, 9:33 AM

A new class, the InteractionSchemeSwitcher has already been introduced in 9ab09bff137b. This class is used as a replacement for the MouseModeSwitcher. It it a simplified and cleaned version which reduces the responsibilities of the class: The service registration has been removed and is now done inside QmitkAbstractMultiWidget and DisplayActionEventBroadcast.
Additionally, the scheme switcher is adjusted to match the new scheme config files (see T24368): There is no difference anymore between a mouse mode and an interaction scheme (see T24368).
The new scheme switcher is used within the QmitkInteractionScheme and is currently tested with the QmitkAbstractMultiWidget.

Has been solved on top of T26476.
This task will be closed if everything is merged to master and tested (wait for D212).

This task / branch was never merged into master; the mentioned BlackSwan task T26476 is still not contained in master / develop.
For reasons of cleanliness I will open a new branch / differential containing just the changes related to this task such that we can merge the new branch into develop and close this task.
@s434n will help me evaluate the relevance / validity of the changes.

kalali added a revision: Restricted Differential Revision.

Deleted branch from rMITK MITK: T24357-Remove-MouseModeSwitcher.