Closed p1p3 closed 1 year ago
Please check if the PR fulfills these requirements
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature: Add a visibility option for the uncertainties in the following tools: Rectangle, Circle & Length.
Uncertainties are always displayed.
What is the new behavior (if this is a feature change)?
Centralise the logic to format the measurements from different tools like Rectangle, Circle & Length.
A new base tool BaseMeasurmentTool that includes the dynamic property displayUncertainties implemented as suggested in this cornerstoneTools issue
BaseMeasurmentTool
displayUncertainties
Uncertainties are shown/hidden depending on the state of the displayUncertainties.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.
Please check if the PR fulfills these requirements
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature: Add a visibility option for the uncertainties in the following tools: Rectangle, Circle & Length.
Uncertainties are always displayed.
What is the new behavior (if this is a feature change)?
Centralise the logic to format the measurements from different tools like Rectangle, Circle & Length.
A new base tool
BaseMeasurmentToolthat includes the dynamic propertydisplayUncertaintiesimplemented as suggested in this cornerstoneTools issueUncertainties are shown/hidden depending on the state of the displayUncertainties.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.