Page MenuHomePhabricator | MITK

Set 'selected' property when using the new selection concept
Open, NormalPublic

Description

Looking at the data manager we find
node->SetSelected(..) in void QmitkDataManagerView::NodeSelectionChanged(const QItemSelection& /*selected*/, const QItemSelection& /*deselected*/).
This sets the selected property of the data node so that other plugins can easily find out if a node has been selected. This is currently used in T25483.
Since we use the new selection concept - also with a focus on a more local selection - we do not maintain the selected property anymore (apart from the data manager).

We should investigate if this property is still needed / used by other plugins and if so how we want to maintain the selected property using the new selection concept.

Event Timeline

kalali triaged this task as Normal priority.Jan 2 2019, 6:28 PM
kalali created this task.
kalali removed kalali as the assignee of this task.Jun 14 2019, 4:02 PM

This would be necessary if we want to use the render window manager with the mxnmultiwidget and the mentioned task (T25483).

floca added a comment.Mon, Aug 12, 3:22 PM

This would be necessary if we want to use the render window manager with the mxnmultiwidget and the mentioned task (T25483).

This is only true if we use the old, non local level window widget. We have also differnt other issues with this widget, therefor I would include "skipping it and using other means/widgets for level window interaction" also in our considerations.

I came across this as I re-added the level window slider to the MxNMultiWidget (we didn't include this before as we used the PACS mode inside the MxNMultiWidget - which allows to change the level window via mouse moving). Now our user wants to use the level-window-presets available from the context menu of the level window slider.

floca added a comment.Wed, Aug 21, 7:10 PM

OK. But we definetly should talk about it before it is approached.