Skip to content

Commit 377be0f

Browse files
committed
zoomLevels didn't work when only a single zoom level was provided
1 parent 8ddee94 commit 377be0f

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

inst/htmlwidgets/leaflet.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1969,8 +1969,9 @@ methods.setGroupOptions = function (group, options) {
19691969

19701970
_jquery2.default.each((0, _util.asArray)(group), function (i, g) {
19711971
var layer = _this8.layerManager.getLayerGroup(g, true);
1972-
if (options.zoomLevels) {
1973-
layer.zoomLevels = options.zoomLevels;
1972+
// This slightly tortured check is because 0 is a valid value for zoomLevels
1973+
if (typeof options.zoomLevels !== "undefined" && options.zoomLevels !== null) {
1974+
layer.zoomLevels = (0, _util.asArray)(options.zoomLevels);
19741975
}
19751976
});
19761977
this.showHideGroupsOnZoom();

javascript/src/methods.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -874,8 +874,9 @@ methods.showGroup = function(group) {
874874
methods.setGroupOptions = function(group, options) {
875875
$.each(asArray(group), (i, g) => {
876876
let layer = this.layerManager.getLayerGroup(g, true);
877-
if (options.zoomLevels) {
878-
layer.zoomLevels = options.zoomLevels;
877+
// This slightly tortured check is because 0 is a valid value for zoomLevels
878+
if (typeof(options.zoomLevels) !== "undefined" && options.zoomLevels !== null) {
879+
layer.zoomLevels = asArray(options.zoomLevels);
879880
}
880881
});
881882
this.showHideGroupsOnZoom();

0 commit comments

Comments
 (0)