New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ENH: Support restoring SliceViewAnnotations enabled state #7176
ENH: Support restoring SliceViewAnnotations enabled state #7176
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
ff4bf23
to
4a149d2
Compare
CheckListStatus: Updated topic works as expected
|
Slice view annotations is a mess. It was a quick prototype that got integrated to have this important feature in Slicer, but the design is really fragile and limited. We would need to rebuild it from scratch, using proper MRML-based design and flexibility and extensibility in mind, as overlay on image corners is very important space for displaying information. |
Many viewers also use these as UI controls/widgets too (e.g. control reslice rotation or window/level tools by clicking on the corresponding corner annotation can be logical and discoverable). |
4a149d2
to
d11a1c6
Compare
CheckListStatus: Updated topic works as expected ✅
|
Also simplify setting of checkbox state in SliceViewAnnotations
d11a1c6
to
6822a6f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Also simplify setting of checkbox state in SliceViewAnnotations
This pull request was created while working on this issue: