cartodb/Leaflet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add oldLatlng parameter to Marker move event, will allow MarkerCluster to support moving markers.

Browse Source
This commit is contained in:
danzel 2014-01-29 11:33:08 +13:00
parent 02db8b4e1a
commit e2d9a97836
2 changed files with 25 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
spec/suites/layer/marker/MarkerSpec.js
View File

@ -91,4 +91,27 @@ describe("Marker", function () {
expect(marker._shadow.parentNode).to.be(map._panes.shadowPane);
});
});
describe("#setLatLng", function () {
it("fires a move event", function () {
var marker = new L.Marker([0, 0], { icon: icon1 });
map.addLayer(marker);
var beforeLatLng = marker._latlng;
var afterLatLng = new L.LatLng(1, 2);
var eventArgs = null;
marker.on('move', function (e) {
eventArgs = e;
});
marker.setLatLng(afterLatLng);
expect(eventArgs).to.not.be(null);
expect(eventArgs.oldLatlng).to.be(beforeLatLng);
expect(eventArgs.latlng).to.be(afterLatLng);
expect(marker.getLatLng()).to.be(afterLatLng);
});
});
});

3
src/layer/marker/Marker.js
View File

@ -55,9 +55,10 @@ L.Marker = L.Layer.extend({
},
setLatLng: function (latlng) {
var oldLatlng = this._latlng;
this._latlng = L.latLng(latlng);
this.update();
return this.fire('move', {latlng: this._latlng});
return this.fire('move', { oldLatlng: oldLatlng, latlng: this._latlng });
},
setZIndexOffset: function (offset) {