Page MenuHomePhabricator

Missing transition while rotating and zooming planes
Closed, WontfixPublic

Description

If the crosshair is in rotation mode and the cursor switches to rotation mode it doesn't switch back if another button is clicked while rotating the planes, e.g. right mouse for zooming. The endrotation event is missing in this case.

Event Timeline

User thomass has pushed new remote branch:

bug-19462-MissingTransistionWhenRotating

Actual Behaviour
While being in crosshair rotation mode, pressing another second button causes the interactor to remain in that previous state and the cursor icon to freeze.

Expected behavior
Pressing another button should lead to a state from which other actions can be reached.

Cause of the bug
Missing transistion in state "Rotation" in DisplayInteraction.xml

Proposed solution
An additional transistion was added to cope with the particular case when a second button is pressed during crosshair mode.

Affected classes
.xml file /Core/resource/Interactions/DisplayInteraction.xml was modified.

How will the bugfix get tested?
Behavior has been observed during user interaction.

kislinsk triaged this task as Normal priority.Aug 10 2016, 3:26 AM
kislinsk edited projects, added MITK (2016-11); removed MITK.
kislinsk added a subscriber: kislinsk.

Unfortunately, this revision was never reviewed. @thomass Would you mind to use arc diff to submit a code review "again"? I'm happy to assist you installing arc if you're not sure what you need to do. I also referenced a quick guide at the bottom of our Phabricator Migration page. Thank you!

kislinsk lowered the priority of this task from Normal to Low.Oct 28 2016, 3:11 PM
kislinsk edited projects, added MITK; removed MITK (2016-11).
kislinsk removed thomass as the assignee of this task.
kislinsk added a project: Auto-closed.
kislinsk added a subscriber: thomass.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs