MantisBT - VTK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0007512VTK(No Category)public2008-08-20 10:422008-09-12 11:20
ReporterVince Magnotta 
Assigned ToKarthik Krishnan 
PrioritynormalSeveritytweakReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Project
Type
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.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-08-20 10:42Vince MagnottaNew Issue
2008-09-11 15:14Karthik KrishnanNote Added: 0013443
2008-09-11 15:15Karthik KrishnanAssigned To => Karthik Krishnan
2008-09-11 15:15Karthik KrishnanStatusbacklog => @80@
2008-09-11 15:15Karthik KrishnanResolutionopen => fixed
2008-09-11 15:19Karthik KrishnanStatus@80@ => @20@
2008-09-11 15:19Karthik KrishnanResolutionfixed => reopened
2008-09-11 15:19Karthik KrishnanNote Added: 0013444
2008-09-11 15:19Karthik KrishnanStatus@20@ => @80@
2008-09-11 15:19Karthik KrishnanResolutionreopened => fixed
2008-09-12 11:20David ColeStatus@80@ => closed
2008-09-12 11:20David ColeNote Added: 0013451
2011-06-16 13:11Zack GalbreathCategory => (No Category)

Notes
(0013443)
Karthik Krishnan   
2008-09-11 15:14   
Fixed in CVS.
   vtkPlaneWidget.cxx
   vtkPlaneWidget.h


/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.
----------------------------
(0013444)
Karthik Krishnan   
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.
(0013451)
David Cole   
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: 1.2.32.1; previous revision: 1.2
    /cvsroot/VTK/VTK/Widgets/vtkPlaneWidget.h,v <-- Widgets/vtkPlaneWidget.h
    new revision: 1.1.58.1; previous revision: 1.1
    /cvsroot/VTK/VTKData/Baseline/Widgets/TestPlaneWidget.png,v <-- Baseline/Widgets/TestPlaneWidget.png
    new revision: 1.1.8.1; previous revision: 1.1