VladPolskiy/leaflet
1
0
Fork 0
mirror of https://github.com/Leaflet/Leaflet.git synced 2025-08-20 14:34:38 +00:00
Code Issues Packages Projects Releases 1 Wiki Activity

Update the Layers control when layers are added or removed

Browse Source
This commit is contained in:
Tom Hughes
2012-11-22 12:12:31 +00:00
parent b5c0a5f5f9
commit 0b2a31c1fe
2 changed files with 56 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
spec/suites/control/Control.LayersSpec.js
View File

@ -29,4 +29,37 @@ describe("Control.Layers", function () {
expect(spy).not.toHaveBeenCalled(); expect(spy).not.toHaveBeenCalled();
}); });
}); });
describe("updates", function () {
beforeEach(function () {
map.setView([0, 0], 14);
});
it("when an included layer is addded or removed", function () {
var baseLayer = L.tileLayer(),
overlay = L.marker([0, 0]),
layers = L.control.layers({"Base": baseLayer}, {"Overlay": overlay}).addTo(map);
spyOn(layers, '_update').andCallThrough();
map.addLayer(overlay);
map.removeLayer(overlay);
expect(layers._update).toHaveBeenCalled();
expect(layers._update.callCount).toEqual(2);
});
it("not when a non-included layer is added or removed", function () {
var baseLayer = L.tileLayer(),
overlay = L.marker([0, 0]),
layers = L.control.layers({"Base": baseLayer}).addTo(map);
spyOn(layers, '_update').andCallThrough();
map.addLayer(overlay);
map.removeLayer(overlay);
expect(layers._update).not.toHaveBeenCalled();
});
});
}); });

23
src/control/Control.Layers.js
View File

@ -10,6 +10,7 @@ L.Control.Layers = L.Control.extend({
this._layers = {}; this._layers = {};
this._lastZIndex = 0; this._lastZIndex = 0;
this._handlingClick = false;
for (var i in baseLayers) { for (var i in baseLayers) {
if (baseLayers.hasOwnProperty(i)) { if (baseLayers.hasOwnProperty(i)) {
@ -28,9 +29,19 @@ L.Control.Layers = L.Control.extend({
this._initLayout(); this._initLayout();
this._update(); this._update();
map
.on('layeradd', this._onLayerChange, this)
.on('layerremove', this._onLayerChange, this);
return this._container; return this._container;
}, },
onRemove: function (map) {
map
.off('layeradd', this._onLayerChange)
.off('layerremove', this._onLayerChange);
},
addBaseLayer: function (layer, name) { addBaseLayer: function (layer, name) {
this._addLayer(layer, name); this._addLayer(layer, name);
this._update(); this._update();
@ -133,6 +144,14 @@ L.Control.Layers = L.Control.extend({
this._separator.style.display = (overlaysPresent && baseLayersPresent ? '' : 'none'); this._separator.style.display = (overlaysPresent && baseLayersPresent ? '' : 'none');
}, },
_onLayerChange: function (e) {
var id = L.stamp(e.layer);
if (this._layers[id] && !this._handlingClick) {
this._update();
}
},
// IE7 bugs out if you create a radio dynamically, so you have to do it this hacky way (see http://bit.ly/PqYLBe) // IE7 bugs out if you create a radio dynamically, so you have to do it this hacky way (see http://bit.ly/PqYLBe)
_createRadioElement: function (name, checked) { _createRadioElement: function (name, checked) {
@ -184,6 +203,8 @@ L.Control.Layers = L.Control.extend({
inputsLen = inputs.length, inputsLen = inputs.length,
baseLayer; baseLayer;
this._handlingClick = true;
for (i = 0; i < inputsLen; i++) { for (i = 0; i < inputsLen; i++) {
input = inputs[i]; input = inputs[i];
obj = this._layers[input.layerId]; obj = this._layers[input.layerId];
@ -201,6 +222,8 @@ L.Control.Layers = L.Control.extend({
if (baseLayer) { if (baseLayer) {
this._map.fire('baselayerchange', {layer: baseLayer}); this._map.fire('baselayerchange', {layer: baseLayer});
} }
this._handlingClick = false;
}, },
_expand: function () { _expand: function () {