View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0003262Slicer4Module DataProbepublic2013-07-31 06:172018-10-31 15:24
Reporterfedorov Assigned Topieper  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target VersionbacklogFixed in VersionSlicer 4.11.0 
Summary0003262: Data probe panel is not updated when scrolling volume slices using keyboard
Description

Need to move the mouse to trigger the update. Is this intentional behavior?

TagsNo tags attached.

Activities

pieper

pieper

2013-08-01 12:09

administrator   ~0009345

Please try r22253
fedorov

fedorov

2013-08-03 04:58

developer   ~0009376

Perfect - thanks a lot!
jcfr

jcfr

2015-09-14 21:14

administrator   ~0013271

Last edited: 2015-09-14 21:17

Re-opening the issue. Commit r23387 (ENH: Added mouse cursor position to vtkMRMLCrosshairNode.) most likely "broke" this.

Note that at the time of commit r23387, the magnified slice view had been removed. It was reintroduced only later by Alex in r24284 (added magnified slice view into DataProbe panel.)
lassoan

lassoan

2018-05-30 00:03

developer   ~0015795

Retarget stale issues to backlog.
jamesobutler

jamesobutler

2018-10-30 22:53

reporter   ~0016181

A user posted on the forum about data probe not updating on mouse wheel scrolling which is likely the same issue as specified here which is data probe not updating when scrolling using the keyboard.

See https://discourse.slicer.org/t/change-data-probe-to-view-mip-hu/4516
lassoan

lassoan

2018-10-30 23:32

developer   ~0016182

DataProbe should also observe slice nodes and update content if the current slice node is modified.
pieper

pieper

2018-10-31 11:29

administrator   ~0016183

Probably the CrosshairNode should be firing anytime the SliceNode is modified and the DataProbe should stay as-is.
lassoan

lassoan

2018-10-31 14:19

developer   ~0016184

Good point. I would not touch the data probe (it would need a complete rewrite), but if the issue can be solved in the interactor then it may be a simple and clear. I'll look into it.
lassoan

lassoan

2018-10-31 15:24

developer   ~0016185

Fixed in r27527

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file 4d2dd723

2013-08-01 16:09:31

pieper

Details Diff 		BUG: 0003262 update data probe on non-mouse-move events

Allow key strokes and other events that change the slice display
to trigger updates to the DataProbe window. Only update the display
for the sliceView that matches the one most recently entered by
the mouse pointer.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22253 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/DataProbe/DataProbe.py Diff File

Import 2017-06-07 23:51:09: master 27de5a06

2015-09-15 18:44:32

jcfr

Details Diff 		STYLE: DataProbe: Remove unused code setting 'currentLayoutName' ivar

This commit removes code initially introduced in r22253 (BUG: 0003262
update data probe on non-mouse-move events) that later became obsolete
with r23387 (ENH: Added mouse cursor position to vtkMRMLCrosshairNode.)

From: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24547 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/DataProbe/DataProbe.py Diff File

Issue History

Date Modified Username Field Change
2013-07-31 06:17 fedorov New Issue
2013-07-31 06:17 fedorov Status new => assigned
2013-07-31 06:17 fedorov Assigned To => pieper
2013-08-01 12:07 pieper Target Version => Slicer 4.3.0
2013-08-01 12:09 pieper Note Added: 0009345
2013-08-01 12:09 pieper Status assigned => resolved
2013-08-01 12:09 pieper Fixed in Version => Slicer 4.3.0
2013-08-01 12:09 pieper Resolution open => fixed
2013-08-03 04:58 fedorov Note Added: 0009376
2013-08-03 04:58 fedorov Status resolved => closed
2015-09-14 21:13 jcfr Status closed => acknowledged
2015-09-14 21:13 jcfr Target Version Slicer 4.3.0 =>
2015-09-14 21:14 jcfr Note Added: 0013271
2015-09-14 21:17 jcfr Note Edited: 0013271
2015-09-14 21:17 jcfr Note Edited: 0013271
2017-06-07 23:27 pieper Changeset attached => Slicer 2145-support-for-installing-extension-from-file 4d2dd723
2017-06-10 08:51 jcfr Changeset attached => Slicer master 27de5a06
2018-05-30 00:03 lassoan Target Version => backlog
2018-05-30 00:03 lassoan Note Added: 0015795
2018-10-30 22:53 jamesobutler Note Added: 0016181
2018-10-30 23:32 lassoan Note Added: 0016182
2018-10-31 11:29 pieper Note Added: 0016183
2018-10-31 14:19 lassoan Note Added: 0016184
2018-10-31 15:24 lassoan Status acknowledged => resolved
2018-10-31 15:24 lassoan Fixed in Version Slicer 4.3.0 => Slicer 4.11.0
2018-10-31 15:24 lassoan Note Added: 0016185