Skip to content
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

BUG: Fix localization and inconsistent state in ctkModalityWidget #1080

Merged
merged 1 commit into from
Apr 12, 2023

Commits on Apr 12, 2023

  1. BUG: Fix localization and inconsistent state in ctkModalityWidget

    There were several issues with the imaging modality selector widget (ctkModalityWidget):
    - when imaging modality abbreviations (CT, MRI, XA, ...) were localized then the widget did not work at all, because the checkbox text was used to identify the checkboxes
    - widget state was stored redundantly in SelectedModalities and VisibleModalities variables and the checkbox properties; and these were often inconsistent
    
    Fixed by using Qt object names to identify checkboxes; and removing SelectedModalities and VisibleModalities variables (store widget state in the checkbox properties).
    lassoan committed Apr 12, 2023
    Configuration menu
    Copy the full SHA
    d4977ad View commit details
    Browse the repository at this point in the history