View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0007512VTK(No Category)public2008-08-20 10:422008-09-12 11:20
ReporterVince Magnotta 
Assigned ToKarthik Krishnan 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0007512: vtkPlaneWidget Sphere Resizing
DescriptionIf make a call to SetHandleSize() for this widget, the Handles are originally sized correctly. However, Each of the On*ButtonUp methods make a call to SizeHandles(). This resets the handle size, eventhough I have previously called to SetHandleSize(). I have commented out the call to SizeHandles() and this can be eliminated while retaining the expected behaviour.
TagsNo tags attached.
Attached Files


Karthik Krishnan (reporter)
2008-09-11 15:14

Fixed in CVS.

/cvsroot/VTK/VTK/Widgets/vtkPlaneWidget.cxx,v <-- vtkPlaneWidget.cxx
new revision: 1.3; previous revision: 1.2
/cvsroot/VTK/VTK/Widgets/vtkPlaneWidget.h,v <-- vtkPlaneWidget.h
new revision: 1.2; previous revision: 1.1
/cvsroot/VTK/VTKData/Baseline/Widgets/TestPlaneWidget.png,v <-- TestPlaneWidget.png
new revision: 1.2; previous revision: 1.1

revision 1.2
date: 2008-09-11 15:11:28 -0400; author: karthik; state: Exp; lines: +3 -0; commitid: oeoI9WOj7qQm5oit;
BUG: Fixing bug 0007512.
  The HandleSize is determined dynamically based on the window size, focal point etc. However when the handle size is set by the user, typically before the widget is enabled, the camera parameters are not yet available, since the widget hasnt been enabled yet. The handle size in this case is fudged to a factor of the user specified size. Once the widget is enabled, the handle size suddenly changes. The transition is ANNOYING.

The new way checks if the widget is being enabled for the first time and if so, sets a factor that preserves the old handle size on the first render, making the transition more gradual.
Karthik Krishnan (reporter)
2008-09-11 15:19

As an additional note, to verify that this works,

On TestPlaneWidget.cxx, add the line

  planeWidget->SetHandleSize( 0.1 );

before planeWidget->PlanceWidget();

The handle size should not jump suddenly at any point.
David Cole (developer)
2008-09-12 11:20

Fix also merged to the VTK-5-2 branch:

    /cvsroot/VTK/VTK/Widgets/vtkPlaneWidget.cxx,v <-- Widgets/vtkPlaneWidget.cxx
    new revision:; previous revision: 1.2
    /cvsroot/VTK/VTK/Widgets/vtkPlaneWidget.h,v <-- Widgets/vtkPlaneWidget.h
    new revision:; previous revision: 1.1
    /cvsroot/VTK/VTKData/Baseline/Widgets/TestPlaneWidget.png,v <-- Baseline/Widgets/TestPlaneWidget.png
    new revision:; previous revision: 1.1

 Issue History
Date Modified Username Field Change
2008-08-20 10:42 Vince Magnotta New Issue
2008-09-11 15:14 Karthik Krishnan Note Added: 0013443
2008-09-11 15:15 Karthik Krishnan Assigned To => Karthik Krishnan
2008-09-11 15:15 Karthik Krishnan Status backlog => @80@
2008-09-11 15:15 Karthik Krishnan Resolution open => fixed
2008-09-11 15:19 Karthik Krishnan Status @80@ => @20@
2008-09-11 15:19 Karthik Krishnan Resolution fixed => reopened
2008-09-11 15:19 Karthik Krishnan Note Added: 0013444
2008-09-11 15:19 Karthik Krishnan Status @20@ => @80@
2008-09-11 15:19 Karthik Krishnan Resolution reopened => fixed
2008-09-12 11:20 David Cole Status @80@ => closed
2008-09-12 11:20 David Cole Note Added: 0013451
2011-06-16 13:11 Zack Galbreath Category => (No Category)

Copyright © 2000 - 2018 MantisBT Team