Skip to content

BUG: Restore maximization of view on double click event#7442

Merged
jcfr merged 3 commits intoSlicer:mainfrom
jcfr:fix-interactor-style-event-observation-regression
Dec 4, 2023
Merged

BUG: Restore maximization of view on double click event#7442
jcfr merged 3 commits intoSlicer:mainfrom
jcfr:fix-interactor-style-event-observation-regression

Conversation

@jcfr
Copy link
Member

@jcfr jcfr commented Dec 3, 2023

This commit fixes a regression introduced in 1cc3b2c (ENH: Improve Event Delegation in qMRMLThreeDView and qMRMLSliceView) adding missing observers.

@jcfr jcfr linked an issue Dec 3, 2023 that may be closed by this pull request
@jcfr jcfr requested review from jamesobutler and pieper December 3, 2023 22:37
@jcfr jcfr added the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Dec 3, 2023
jcfr added 3 commits December 3, 2023 18:05
This commit fixes a regression introduced in 1cc3b2c (ENH: Improve Event
Delegation in qMRMLThreeDView and qMRMLSliceView) adding missing observers.
…/pan

This commit fixes a regression introduced in 1cc3b2c (ENH: Improve Event
Delegation in qMRMLThreeDView and qMRMLSliceView) adding missing observers.
@jcfr jcfr force-pushed the fix-interactor-style-event-observation-regression branch from a7c2cb5 to 8c5c03c Compare December 3, 2023 23:10
Copy link
Member

@pieper pieper left a comment

Choose a reason for hiding this comment

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

LGTM 👍 thanks

@jcfr jcfr merged commit 017f277 into Slicer:main Dec 4, 2023
@jcfr jcfr deleted the fix-interactor-style-event-observation-regression branch December 4, 2023 14:13
@lassoan
Copy link
Contributor

lassoan commented Dec 4, 2023

Thank you, I can confirm confirm that this also fixes touchscreen interactions on Windows that got broken as well.

@lassoan
Copy link
Contributor

lassoan commented Dec 4, 2023

@jcfr would it be possible to create a 5.6.1 patch release within a few days?

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.

Doubleclick to maximize sliceViews and threeDViews stopped working

3 participants