From 29233213f0cf0ea61cb47a9d639b9ebb1dc69273 Mon Sep 17 00:00:00 2001 From: Joey Baker Date: Sat, 12 Oct 2013 16:24:36 -0700 Subject: [PATCH] EditToolbar now sets it's state correctly Properly determines if it has layers, and then sets the button's disabled class. fixes #202 --- src/edit/EditToolbar.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/edit/EditToolbar.js b/src/edit/EditToolbar.js index f576ad1af..e975b4937 100644 --- a/src/edit/EditToolbar.js +++ b/src/edit/EditToolbar.js @@ -123,13 +123,14 @@ L.EditToolbar = L.Toolbar.extend({ _checkDisabled: function () { var featureGroup = this.options.featureGroup, - hasLayers = featureGroup.getLayers().length === 0, + hasLayers = featureGroup.getLayers().length !== 0, button; if (this.options.edit) { button = this._modes[L.EditToolbar.Edit.TYPE].button; - L.DomUtil.toggleClass(button, 'leaflet-disabled'); + if (hasLayers) { L.DomUtil.removeClass(button, 'leaflet-disabled'); } + else { L.DomUtil.addClass(button, 'leaflet-disabled'); } button.setAttribute( 'title', @@ -142,7 +143,8 @@ L.EditToolbar = L.Toolbar.extend({ if (this.options.remove) { button = this._modes[L.EditToolbar.Delete.TYPE].button; - L.DomUtil.toggleClass(button, 'leaflet-disabled'); + if (hasLayers) { L.DomUtil.removeClass(button, 'leaflet-disabled'); } + else { L.DomUtil.addClass(button, 'leaflet-disabled'); } button.setAttribute( 'title', @@ -164,4 +166,4 @@ if (!L.DomUtil.toggleClass) { } } }); -} \ No newline at end of file +}