Larry, Before you commit, can you replace duplicate method calls such as getPanel().getViewport() with a local variable and make the method call once. There are two reasons for this.
1. There is less code to read on each line which makes working out what is going on easier. 2. When debuging if you follow a one method call per line rule it makes stepping into a method easier you don't have to go up and down into each method call. Cheers, Paul Larry Becker wrote: > Well, I guess I'll commit the change on the strength or Paul and > Sunburn's recommendation. We can always back out the change if we > don't like it. > > Larry > > On 8/17/07, Larry Becker <[EMAIL PROTECTED]> wrote: > >> Hi Paul, >> >> Some of the functionality you are talking about is already present >> in my new ZoomRealTime tool, which pans and zooms continuously in real >> time. Perhaps I should port it over (it just needs >> internationalization) before doing any more tweaks to Mouse Wheel >> Zoom, which I thought was pretty cool when I first got it working, but >> now seems a little tired in comparison with ZoomRealTime. >> >> I do agree that your #2 is more useful, now that I have tried it. >> I'll wait for more feedback from others before committing the change. >> >> regards, >> Larry >> >> >> >> On 8/17/07, Paul Austin <[EMAIL PROTECTED]> wrote: >> >>> Hi Larry, >>> >>> Normally I hate people messing with my mouse pointer but in this case I >>> would find it useful. >>> >>> Here are the three options for mouse wheel zooming that I see. >>> >>> 1. Zoom in and out of the current viewport center (not that useful in >>> my view). >>> 2. Zoom in and out at the current mouse location (google maps does >>> this) (default option) >>> 3. Zoom in and out of the current mouse location but center the >>> viewport and mouse pointer at that location (advanced option while >>> holding down the shift key) >>> >>> Another thing I'd like to see is an integrated pan and zoom mode, where >>> if you double click it zooms but when you drag it pans, much like google >>> maps does. This could be integrated into the current pan tool. I'd still >>> like the BBOX zoom functionality from the zoom tool. >>> >>> Paul >>> >>> ------------------------------------------------------------------------- >>> This SF.net email is sponsored by: Splunk Inc. >>> Still grepping through log files to find problems? Stop. >>> Now Search log events and configuration files using AJAX and a browser. >>> Download your FREE copy of Splunk now >> http://get.splunk.com/ >>> _______________________________________________ >>> Jump-pilot-devel mailing list >>> Jump-pilot-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >>> >>> >> -- >> http://amusingprogrammer.blogspot.com/ >> >> > > > ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
