MantisBT - VTK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013707VTK(No Category)public2012-12-06 08:572014-10-02 10:45
ReporterGiuseppe D'Angelo 
Assigned ToMarcus D. Hanwell 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version5.8.0 
Target VersionFixed in Version 
ProjectTBD
Typeincorrect functionality
Summary0013707: vtkXOpenGLRenderWindow doesn't properly reset to the default cursor
DescriptionVTK 5.10.1, in vtkXOpenGLRenderWindow.cxx:

   1985 if (shape == VTK_CURSOR_DEFAULT)
   1986 {
   1987 XUndefineCursor(this->DisplayId,this->WindowId);
   1988 return;
   1989 }

I think this is wrong. This should reset the cursor to the one was set with SetCurrentCursor. Doing so will just switch to the parent's cursor instead.

As a test, set a non-arrow cursor with SetCurrentCursor, then add a widget which changes the mouse cursor (f.i. a vtkScalarBarWidget). When moving the mouse outside the widget, it'll reset the cursor to a plain arrow.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-12-06 08:57Giuseppe D'AngeloNew Issue
2013-06-25 13:48Dave DeMarleNote Added: 0031058
2014-10-02 10:10Berk GeveciAssigned To => Marcus D. Hanwell
2014-10-02 10:23Marcus D. HanwellNote Added: 0033437
2014-10-02 10:23Marcus D. HanwellStatusbacklog => active development
2014-10-02 10:44Marcus D. HanwellNote Added: 0033446
2014-10-02 10:44Marcus D. HanwellStatusactive development => backlog
2014-10-02 10:45Marcus D. HanwellNote Added: 0033447
2014-10-02 10:45Marcus D. HanwellStatusbacklog => closed
2014-10-02 10:45Marcus D. HanwellResolutionopen => fixed

Notes
(0031058)
Dave DeMarle   
2013-06-25 13:48   
Giuseppe can you submit the patch via gerrit to facilitate automated testing and review? For details on how to do so see http://www.vtk.org/Wiki/VTK/Git/Develop [^]
(0033437)
Marcus D. Hanwell   
2014-10-02 10:23   
Looking at this now, should be pretty simply to fix up.
(0033446)
Marcus D. Hanwell   
2014-10-02 10:44   
Line 1958 in the current master has what looks like your code snippet, this is the only occurrence of VTK_CURSOR_DEFAULT, this call is in the SetCurrentCursor method - either that was changed in the meantime, or I need more information on the expected behavior.
(0033447)
Marcus D. Hanwell   
2014-10-02 10:45   
Please reopen if you can provide more information.