Ticket #3017 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

add vertexremoved event

Reported by: ahocevar Owned by: tschaub
Priority: minor Milestone: 2.11 Release
Component: Control.ModifyFeature Version: 2.10
Keywords: Cc:
State: Complete

Description

Deleting vertices currently only triggers the featuremodified event. If applications want to know when a vertex was removed, it would be convenient to have a vertexremoved event.

Attachments

openlayers-3017.patch Download (4.4 KB) - added by ahocevar 2 years ago.

Change History

Changed 2 years ago by ahocevar

Changed 2 years ago by ahocevar

  • state set to Review

Patch with tests. The vertexremoved event has the same properties in its argument as the vertexmodified event.

Tests pass in Safari5. Thanks for any review.

Changed 2 years ago by bartvde

  • state changed from Review to Commit

Looks good to me Andreas, Control.ModifyFeature and Layer.Vector tests also pass in FF 3.6 and IE8.

Changed 2 years ago by ahocevar

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

in w/ r11043

Note: See TracTickets for help on using tickets.