View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0013707

VTK

(No Category)

public

2012-12-06 08:57

2014-10-02 10:45

Reporter

Giuseppe D'Angelo

Assigned To

Marcus D. Hanwell

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

5.8.0

Target Version

Fixed in Version

Summary

0013707: vtkXOpenGLRenderWindow doesn't properly reset to the default cursor

Description

VTK 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.

Tags

No tags attached.

Project

TBD

Type

incorrect functionality

Attached Files

Relationships

Notes
(0031058) Dave DeMarle (administrator) 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 (developer) 2014-10-02 10:23	Looking at this now, should be pretty simply to fix up.

(0033446) Marcus D. Hanwell (developer) 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 (developer) 2014-10-02 10:45	Please reopen if you can provide more information.

Issue History
Date Modified	Username	Field	Change
2012-12-06 08:57	Giuseppe D'Angelo	New Issue
2013-06-25 13:48	Dave DeMarle	Note Added: 0031058
2014-10-02 10:10	Berk Geveci	Assigned To	=> Marcus D. Hanwell
2014-10-02 10:23	Marcus D. Hanwell	Note Added: 0033437
2014-10-02 10:23	Marcus D. Hanwell	Status	backlog => active development
2014-10-02 10:44	Marcus D. Hanwell	Note Added: 0033446
2014-10-02 10:44	Marcus D. Hanwell	Status	active development => backlog
2014-10-02 10:45	Marcus D. Hanwell	Note Added: 0033447
2014-10-02 10:45	Marcus D. Hanwell	Status	backlog => closed
2014-10-02 10:45	Marcus D. Hanwell	Resolution	open => fixed