As described in T26485, current behavior when activating any segmentation tool is to overwrite the display interaction config with a hardcoded config that doesn't use the left mouse button (DisplayConfigMITKNoCrosshair.xml) and restore the original config when the tool is deactivated. This does not work when the PACS interaction is used, since it is still overwritten by the default config (without crosshair).
To solve this, I propose to switch from setting a new config in mitk::Tool::Activated() to adding it to the current config. This way one does not need to handle all the unchanged interactions, only the ones one wants to change. For example, for segmentation tools, instead of hardcoding the entire config, one can just add a config that only contains a non-functional interaction for the left mouse button. This will block any events of that kind to be handled by the original config, but won't interfer with any others.
Description
Description
Revisions and Commits
Revisions and Commits
rMITK MITK | |||
Restricted Differential Revision | rMITKa3f148c69278 Do not use empty filepath as event config | ||
Restricted Differential Revision | rMITK7ff746b52822 Return early if the given filepath is empty | ||
Restricted Differential Revision | rMITK234cb93299fe Fixed T28667 Generalize overwriting of interaction configs when using tools | ||
Restricted Differential Revision | rMITKaf17da8dcab9 replace setting hardcoded interaction config with adding config containing only… |
Status | Assigned | Task | ||
---|---|---|---|---|
Resolved | s434n | T26485 [Segmentation] PACS mode is active while segmentation is drawn | ||
Resolved | s434n | T28629 [ImageCropper] PACS mode is active while bounding box is drawn | ||
Resolved | s434n | T28667 [Segmentation] Generalize overwriting of interaction configs when using tools |