View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0003222Slicer4Module Annotationspublic2013-07-11 06:402017-06-07 23:27
Reporterkotwicz Assigned Tojcfr  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.2.0 
Target VersionSlicer 4.3.0Fixed in VersionSlicer 4.3.0 
Summary0003222: moving mouse over moving ruler projection causes projection to turn green
Description

running this script:
timer = qt.QTimer()
ruler = slicer.vtkMRMLAnnotationRulerNode()
ruler.Initialize(slicer.mrmlScene)
ruler.SetLocked(1)
ruler.SetPosition2(-12.645108222961426, 3.9290014797188633e-07, 286.9258117675781)
rulerDisplay = ruler.GetDisplayNode()
rulerDisplay.SetColor(1,0,0)
rulerDisplay.SliceProjectionColoredWhenParallelOff()
rulerDisplay.SliceProjectionOn()

def moveRuler():
ruler.SetPosition1(random.randint(30,33),random.randint(30,33),random.randint(30,33))
timer.singleShot(20,moveRuler)

timer.singleShot(500,moveRuler)

  • if you move your mouse over the ruler projection in the yellow or green slice viewer the ruler projection turns green as if it is being selected.. however the ruler is locked and projections shouldn't be selectable
TagsNo tags attached.

Activities

Lchauvin

Lchauvin

2013-07-11 06:47

developer   ~0009010

Thank you.

I submitted the pull request.
Lchauvin

Lchauvin

2013-07-11 08:28

developer   ~0009011

https://github.com/lchauvin/Slicer/tree/3222-ruler-projection-looks-selectable
kotwicz

kotwicz

2013-07-11 08:50

reporter   ~0009013

Great thanks, will these be merged in tonight's nightly?
Lchauvin

Lchauvin

2013-07-11 08:58

developer   ~0009014

I hope so but it depends on JC.
jcfr

jcfr

2013-07-12 20:57

administrator   ~0009036

Fixed in r22177
See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=22177
jcfr

jcfr

2017-06-07 23:27

administrator   ~0014636

Fix committed to 2145-support-for-installing-extension-from-file branch.

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file cd57fdfd

2013-07-13 00:54:23

jcfr

Details Diff 		BUG: Ensure projections do not appear as selectable

Projections are not selectable and shouldn't be, however they
were displayed as if they were selectable, so it could be confusing
for the user. This fix ensure projections are properly displayed to
not look like selectable.

Fixes 0003222

From: Laurent Chauvin <lchauvin@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22177 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Loadable/Annotations/MRMLDM/vtkMRMLAnnotationDisplayableManager.cxx Diff File

Issue History

Date Modified Username Field Change
2013-07-11 06:40 kotwicz New Issue
2013-07-11 06:40 kotwicz Status new => assigned
2013-07-11 06:40 kotwicz Assigned To => nicole
2013-07-11 06:47 Lchauvin Note Added: 0009010
2013-07-11 06:47 Lchauvin Assigned To nicole => Lchauvin
2013-07-11 06:47 Lchauvin Status assigned => confirmed
2013-07-11 08:28 Lchauvin Note Added: 0009011
2013-07-11 08:50 kotwicz Note Added: 0009013
2013-07-11 08:58 Lchauvin Note Added: 0009014
2013-07-12 20:57 jcfr Note Added: 0009036
2013-07-12 20:57 jcfr Status confirmed => resolved
2013-07-12 20:57 jcfr Fixed in Version => Slicer 4.3.0
2013-07-12 20:57 jcfr Resolution open => fixed
2013-07-12 20:57 jcfr Target Version => Slicer 4.3.0
2013-07-16 09:40 kotwicz Status resolved => closed
2017-06-07 23:27 jcfr Changeset attached => Slicer 2145-support-for-installing-extension-from-file cd57fdfd
2017-06-07 23:27 jcfr Note Added: 0014636
2017-06-07 23:27 jcfr Assigned To Lchauvin => jcfr