Skip to content

Commit

Permalink
ENH: Clean up vtkMRMLLinearTransformsDisplayableManager implementation
Browse files Browse the repository at this point in the history
Refactor and combine both 2D and 3D displayable managers for linear transform nodes into a single class.

Since both 2D and 3D displayable managers are now generating the exact same interaction widget, they ended up sharing all of the same code.
To avoid duplication and to reduce maintenance, this commit combines the two displayable managers into a single class.

Re #7570
  • Loading branch information
Sunderlandkyl authored and lassoan committed Feb 17, 2024
1 parent 9c49644 commit 158d74b
Show file tree
Hide file tree
Showing 6 changed files with 198 additions and 1,029 deletions.
3 changes: 1 addition & 2 deletions Modules/Loadable/Transforms/MRMLDM/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@ set(${KIT}_INCLUDE_DIRECTORIES
set(displayable_manager_SRCS
vtkMRML${MODULE_NAME}DisplayableManager2D.cxx
vtkMRML${MODULE_NAME}DisplayableManager3D.cxx
vtkMRMLLinear${MODULE_NAME}DisplayableManager3D.cxx
vtkMRMLLinear${MODULE_NAME}DisplayableManager2D.cxx
vtkMRMLLinear${MODULE_NAME}DisplayableManager.cxx
vtkMRMLTransformHandleWidget.cxx
vtkMRMLTransformHandleWidgetRepresentation.cxx
)
Expand Down

0 comments on commit 158d74b

Please sign in to comment.