View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015689VTK(No Category)public2015-08-27 08:252016-07-07 09:37
ReporterMathieu Westphal 
Assigned ToCory Quammen 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version7.1.0 
Summary0015689: GetAltKey not working
Descriptionthe GetAltKey method is not working, and returns always null.

This is because of
vtkRenderWindowInteractor::SetEventInformation not containing any code concerning the Alt Key.

Of course adding the alt key would need some code adding in every kind of vtkRenderWindowInteractor.
Tagshackaton
ProjectTBD
Typeincorrect functionality
Attached Filestxt file icon addAltKeyInfo.txt [^] (17,986 bytes) 2015-10-21 16:26 [Show Content]

 Relationships

  Notes
(0035079)
Mathieu Westphal (developer)
2015-08-27 08:44

Remarked by David Doria
http://www.paraview.org/Wiki/VTK/Examples/Cxx/Interaction/ShiftAndControl [^]
(0035373)
Alexis Girault (developer)
2015-10-21 16:34
edited on: 2015-10-21 16:36

Changing this implies editing 'SetEvenInformation' methods definition and calls:

https://www.vtk.org/Bug/file/9942/addAltKeyInfo.txt [^]

Adding the 'int alt' parameter to those methods would be easy but would not provide backward compatibility for other projects that use those VTK classes. Would backward compatibility be needed in this case?

(0035376)
Mathieu Westphal (developer)
2015-10-22 03:28

As a reporter, i would say that i stumbled upen this and use GetShiftKey instead.
So fixing this as not critical at all for me.

But anything is better than having an empty method that does nothing without informing the user.

I would advocate for braking backward compability, or removing the GetAltKey method.

Maybe Bill Hoffman or Ken Martin have advice about this.
(0036224)
Cory Quammen (developer)
2016-07-06 15:23

This merge request that is about to go in should fix this issue:

https://gitlab.kitware.com/vtk/vtk/merge_requests/1129/diffs [^]
(0036328)
Cory Quammen (developer)
2016-07-07 09:37

Merged in master.

https://gitlab.kitware.com/vtk/vtk/merge_requests/1129 [^]

 Issue History
Date Modified Username Field Change
2015-08-27 08:25 Mathieu Westphal New Issue
2015-08-27 08:44 Mathieu Westphal Note Added: 0035079
2015-10-19 14:55 Utkarsh Ayachit Tag Attached: hackaton
2015-10-21 11:53 LucasGandel Assigned To => LucasGandel
2015-10-21 16:26 Alexis Girault File Added: addAltKeyInfo.txt
2015-10-21 16:34 Alexis Girault Note Added: 0035373
2015-10-21 16:35 Alexis Girault Note Edited: 0035373
2015-10-21 16:36 Alexis Girault Note Edited: 0035373
2015-10-22 03:28 Mathieu Westphal Note Added: 0035376
2016-07-06 15:23 Cory Quammen Note Added: 0036224
2016-07-06 15:24 Cory Quammen Assigned To LucasGandel => Cory Quammen
2016-07-07 09:37 Cory Quammen Note Added: 0036328
2016-07-07 09:37 Cory Quammen Status backlog => closed
2016-07-07 09:37 Cory Quammen Resolution open => fixed
2016-07-07 09:37 Cory Quammen Fixed in Version => 7.1.0


Copyright © 2000 - 2018 MantisBT Team