Ticket #1326 (closed feature: fixed)

Opened 5 years ago

Last modified 5 years ago

Vector feature specific move() to location method

Reported by: sbenthall Owned by: crschmidt
Priority: critical Milestone: 2.7 Release
Component: Feature.Vector Version: 2.5
Keywords: Cc:
State: Complete

Description

A move() method for Feature.Vector that takes a pixel or LonLat as argument, moves the feature to that location, and redraws.

Attachments

Vector.js.diff Download (0.9 KB) - added by sbenthall 5 years ago.
Patch for Vector.js
Move.diff Download (2.8 KB) - added by sbenthall 5 years ago.
better patch to Vector.js, also a test

Change History

Changed 5 years ago by sbenthall

Patch for Vector.js

Changed 5 years ago by sbenthall

  • state set to Review
  • milestone set to 2.7 Release

Changed 5 years ago by crschmidt

  • state changed from Review to Needs More Work

Needs tests. (Yes, I should have said this earlier, but abandoning our 'test everything' position was a mistake, and I'd like to get away from it, especially for new features.)

Changed 5 years ago by euzuro

  • priority changed from minor to critical

Changed 5 years ago by sbenthall

  • status changed from new to assigned

maybe the layer should be optional on this?

Changed 5 years ago by sbenthall

better patch to Vector.js, also a test

Changed 5 years ago by sbenthall

  • owner changed from sbenthall to crschmidt
  • status changed from assigned to new
  • state changed from Needs More Work to Review

Move.diff has better patch and a test.

Changed 5 years ago by crschmidt

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

r7593

Note: See TracTickets for help on using tickets.