OpenLayers OpenLayers

Ticket #1678 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

add a moveend event to layer

Reported by: elemoine Assigned to:
Priority: minor Milestone: 2.7 Release
Component: Layer Version: SVN
Keywords: Cc:
State: Complete

Description

We already have a moveend event at the map level. This ticket suggest adding one at the layer level. This will be useful to strategies (from the vector behavior work). Without this new event strategies must make sure the layer is in range and visible before going ahead and fetch new features. This isn't good and makes the coding of strategies harder, which we don't want.

Attachments

patch-1678-r7754-A0.diff (1.4 kB) - added by elemoine on 08/22/08 03:17:11.

Change History

08/14/08 09:30:20 changed by elemoine

  • state set to Review.

Unit tests pass on FF2, FF3 and IE7.

08/22/08 03:17:11 changed by elemoine

  • attachment patch-1678-r7754-A0.diff added.

08/25/08 20:15:48 changed by euzuro

  • owner deleted.

I don't think I'm the right person to review this ticket.

Tim or Andreas, can you take a look at this?

(follow-up: ↓ 4 ) 08/26/08 18:21:07 changed by ahocevar

  • state changed from Review to Commit.

This cannot hurt, it does not break anything. And if it is needed for the protocols, go ahead and commit.

(in reply to: ↑ 3 ) 08/26/08 18:22:35 changed by ahocevar

Replying to ahocevar:

This cannot hurt, it does not break anything. And if it is needed for the protocols, go ahead and commit.

I meant to say "... needed for the strategies ...", not "protocols".

08/27/08 01:27:22 changed by elemoine

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

(In [7874]) Add a moveend event to layer. By registering to that event (instead of that at the map level) strategies need to check that the layer is in range and active before fetching new features. r=ahocevar (closes #1678)