Thanks Martin, Changes now checked into svn/trunk. I changed the name of the method to setScrollWheelZoomDelta, and added a get method as well.
On Tue, Mar 10, 2009 at 11:05 PM, Martin Beckett <[email protected]> wrote: > The wheel direction was originally the other way around (ie reeling in the > model toward you) but I discussed the code with Skylark(Jean-Sébastien Guay) > and his code suggestion was the opposite so I submitted that. My guess is that the behaviour one will feel is natural will be entirely down to what other apps you are used to so no one default will be appropriate, so having the delta user definable is very useful. > I was also looking at adding calls to the viewer to do zoom in/out/full > directly from my gui. Is it worth adding calls to TrackballManipulator to > zoom in/out rather than calling get/set Distance? Would this methods just do a setDistance(getDistance()*(1.0+zoomDelta)? If so then they aren't quite something I would view as minimal and complete. > ps. The home() function in TrackballManipulator, to reset a view, takes a > dummy double argument which isn't used - is there a reason for this? This comes from the base class and is there to allow the home method to reset the mouse pointer if it requires it, the driver and flight manipulators all require this. Robert. _______________________________________________ osg-submissions mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
