OpenLayers OpenLayers

Ticket #110 (closed feature: fixed)

Opened 2 years ago

Last modified 10 months ago

animated panning

Reported by: sderle Assigned to: metacarta
Priority: critical Milestone: 2.6 Release
Component: Map Version:
Keywords: Cc:
State: Complete

Description

all of the other mapping services smoothly pan from point (a) to (b) ... why not OL?

Attachments

animatedPanning.patch (4.7 kB) - added by euzuro on 02/22/07 03:42:18.
patchification of code from portugal.. thanks duarte for the leads!
animatedPanning#2.patch (13.1 kB) - added by emanuel on 03/29/07 04:32:06.
fix bug in layers.js; add unit tests
patch-110-r5555-A0.diff (9.3 kB) - added by pgiraud on 12/21/07 12:36:16.
path including a new panTo method, tween and ease methods
patch-110-r5796-B0.diff (9.4 kB) - added by pgiraud on 01/18/08 09:31:15.
110-r5796-C0.diff (6.0 kB) - added by crschmidt on 02/07/08 18:13:24.
110-6099-C1.diff (6.0 kB) - added by crschmidt on 02/08/08 08:26:47.
110-6099-C2.diff (6.0 kB) - added by crschmidt on 02/08/08 08:28:05.
use new 'caller' instead of 'source'
animated_panning.patch (17.4 kB) - added by crschmidt on 02/08/08 09:34:27.

Change History

08/16/06 00:20:49 changed by euzuro

  • milestone deleted.

10/02/06 16:20:17 changed by sderle

  • milestone set to 2.3 Release.

12/28/06 20:13:04 changed by euzuro

  • milestone changed from 2.3 Release to 2.4 Release.

01/03/07 00:38:25 changed by euzuro

From legwork done by Duarte C.:

Here is a link with a working example:
http://sigims.edia.pt/gmaps/final/openlayers_page.html

There is a function that determines how the slide is calculated - if
it's smoother, linear, exponential, etc. At this time it's a power
curve, so initially the movement is bigger and then slows down. This can
be changed by using other curves, as can be seen in moofx
(http://moofx.mad4milk.net/).

<snip>

Finally, you should be aware that I perused an example of animation in
www.hesido.com. It seems free since he encourages others to use the
examples and that's just what I did.

01/03/07 00:39:42 changed by euzuro

and later from Brian L

One thing if I may suggest is to build the slide capability as a property of
the Map.  Thus allowing the slide enablement and effect to be set through
properties of the Map instead of the control.  In this case any standard or
custom control performing a map movement operation (pan, setCenter) can take
advantage of the slide effect when enabled.  Thanks for your contribution!

02/22/07 03:35:52 changed by euzuro

  • keywords set to review.

02/22/07 03:42:18 changed by euzuro

  • attachment animatedPanning.patch added.

patchification of code from portugal.. thanks duarte for the leads!

03/16/07 15:40:14 changed by euzuro

  • milestone changed from 2.4 Release to 2.5 Release.

03/29/07 04:32:06 changed by emanuel

  • attachment animatedPanning#2.patch added.

fix bug in layers.js; add unit tests

03/29/07 04:36:02 changed by emanuel

Demo with animated zooming (#442)

05/13/07 09:28:00 changed by jrf

  • keywords changed from review to review, integrate into trunk.
  • priority changed from minor to blocker.

05/24/07 00:52:45 changed by crschmidt

  • keywords changed from review, integrate into trunk to review.
  • owner changed from sderle to metacarta.
  • priority changed from blocker to critical.

08/03/07 10:35:27 changed by euzuro

  • milestone changed from 2.5 Release to 2.6 Release.

12/15/07 11:21:23 changed by crschmidt

  • state changed from Review to Needs More Work.

This code needs to be cleaned up: There are a couple things in the patch that are totally unrelated to panning/zooming, which should be removed, and it needs updating to trunk.

12/21/07 12:36:16 changed by pgiraud

  • attachment patch-110-r5555-A0.diff added.

path including a new panTo method, tween and ease methods

12/21/07 12:38:01 changed by pgiraud

Proposed patch includes a new panTo method. Tween and Ease object are supposed to be generic and useful for other purposes.

01/18/08 09:30:55 changed by pgiraud

  • state changed from Needs More Work to Review.

Patch updated with issues fixed and panTo used in the overviewmap control.

01/18/08 09:31:15 changed by pgiraud

  • attachment patch-110-r5796-B0.diff added.

01/22/08 14:27:46 changed by pgiraud

  • state changed from Review to Needs More Work.

Enhancement : panTween need to be stopped in case user drags the map manually before the animation is finished.

01/30/08 16:52:19 changed by elemoine

Pierre, I've just applied your patch to trunk and the tween.html example gives me errors:

callback.call is not a function

02/07/08 18:13:24 changed by crschmidt

  • attachment 110-r5796-C0.diff added.

02/07/08 18:14:05 changed by crschmidt

  • state changed from Needs More Work to Review.

Add version of animated panning:

  • With cancel-on-drag support
  • With pan as a wrapper for panTo
  • With a more fun example (dancing map!)

Pierre, I'd love if you could take a look at this. It depends on the patch I've uploaded for 962, and your approval of 1308, which I'm set with.

02/07/08 18:14:43 changed by crschmidt

Oh, and it's available for demo/playing in:

http://dev.openlayers.org/sandbox/crschmidt/animated_panning/

02/07/08 21:22:14 changed by pspencer

There seems to be something wrong with normal panning ... if you just pan the map (in Safari and FF2), the point under the mouse smoothly scrolls to the center after you drop it?

02/07/08 21:23:59 changed by crschmidt

It's a bug in the Click Handler, not the panning code; expect a patch in trunk before the night is out.

02/07/08 22:27:15 changed by crschmidt

(In [6067]) Merge from 6030:HEAD from trunk, primarily to fix up fix to Handler.Click (See #110)

02/08/08 08:26:47 changed by crschmidt

  • attachment 110-6099-C1.diff added.

02/08/08 08:27:08 changed by crschmidt

New title ("Animated Panning", instead of tween)

02/08/08 08:28:05 changed by crschmidt

  • attachment 110-6099-C2.diff added.

use new 'caller' instead of 'source'

02/08/08 09:34:27 changed by crschmidt

  • attachment animated_panning.patch added.

02/08/08 10:40:46 changed by pgiraud

  • state changed from Review to Commit.

This is ready for commit now. All tests pass under FF2 and IE6.

One last note, I would have preferred something named 'animate' instead of 'immediate' and given a true value by default, because it's easier to understand I think. However, you can commit this.

02/08/08 11:28:11 changed by crschmidt

  • status changed from new to closed.
  • state changed from Commit to Complete.
  • resolution set to fixed.

(In [6111]) Add support for animated panning, with most of the work done by Pierre, thx pierre! panTo method now animates when moving. (Closes #110)