Skip to content

ENH: Remove unused API to set/get displayable manager interactor abort flags#7319

Merged
jcfr merged 1 commit intoSlicer:mainfrom
jcfr:remove-unused-dm-interactor-abort-flags-api
Oct 30, 2023
Merged

ENH: Remove unused API to set/get displayable manager interactor abort flags#7319
jcfr merged 1 commit intoSlicer:mainfrom
jcfr:remove-unused-dm-interactor-abort-flags-api

Conversation

@jcfr
Copy link
Member

@jcfr jcfr commented Oct 30, 2023

The InteractorStyleAbort API was originally introduced in 8ded9a0 (ENH: eliminated conflict between crosshair navigation and window level).

Then, it was superseded by the InteractorAbort API in 0b9b339 (BUG: #2629. Window level controls conflicted with crosshair navigation)

Ultimately, the last uses of either the InteractorStyleAbort or InteractorAbort API were removed in dc2c095 (ENH: Improved crosshair usability, added display in 3D view)

…t flags

The InteractorStyleAbort API was originally introduced
in 8ded9a0 (ENH: eliminated conflict between crosshair navigation
and window level).

Then, it was superseded by the InteractorAbort API
in 0b9b339 (BUG: Slicer#2629.  Window level controls conflicted with
crosshair navigation)

Ultimately, the last uses of either the InteractorStyleAbort or InteractorAbort
API were removed in dc2c095 (ENH: Improved crosshair usability, added
display in 3D view)
Copy link
Contributor

@lassoan lassoan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. These are no longer needed.

@jcfr jcfr enabled auto-merge (rebase) October 30, 2023 19:15
@jcfr jcfr merged commit a12d184 into Slicer:main Oct 30, 2023
@jcfr jcfr deleted the remove-unused-dm-interactor-abort-flags-api branch October 30, 2023 19:32
@jcfr jcfr added the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Nov 5, 2023
@jcfr jcfr removed the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants