Skip to content

enh(ui): add the possibility to only allow moving landmarks from a given group

Lucas SCHMIDT requested to merge 1144-enh-ui-add-the-possibility-to-only-allow-moving-landmarks-from-a-given-group into dev

Description

These changes add the possibility to only allow moving landmarks from a given group. It adds the config modify for adaptor::SLandmarks with the possible values all (the default, which allows to modify all landmarks, ignoring current group) and group (which only allows to modify the landmarks belonging to the current group).

Closes #1144 (closed)

How to test it?

  • Launch SightViewer.
  • Create two groups using the landmarks manager and create a few landmarks of each group
  • Check that all the landmarks can be moved whatever the current group is.
  • Check that double-clicking a landmark still works (it should move the slices to the landmark)
  • Modify configs/viz/scene3d/rc/configurations/volumeRendering.xml, add modify="group" in the <config> tag in adaptor::SLandmarks. Don't forget to ninja for the changes to apply.
  • Launch SightViewer.
  • Create two groups using the landmarks manager and create a few landmarks of each group
  • Check that one landmark can be modified only if its group is selected in the landmarks manager. Try with both groups. Also check that if two landmarks of different groups are in the same place, the good one is moved.
  • Check that double-clicking a landmark still works (it should move the slices to the landmark), even if the group of that landmark isn't the current one.
Edited by Lucas SCHMIDT

Merge request reports