View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0007512 | VTK | (No Category) | public | 2008-08-20 10:42 | 2008-09-12 11:20 | ||||
Reporter | Vince Magnotta | ||||||||
Assigned To | Karthik Krishnan | ||||||||
Priority | normal | Severity | tweak | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0007512: vtkPlaneWidget Sphere Resizing | ||||||||
Description | If 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. | ||||||||
Tags | No tags attached. | ||||||||
Project | |||||||||
Type | |||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0013443) Karthik Krishnan (reporter) 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 (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. |
(0013451) 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: 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 |
Notes |
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) |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |