Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

L.EditToolbar._checkDisabled is setting the leaflet-disabled class incorrectly #202

Closed
markgibbons25 opened this issue Oct 9, 2013 · 3 comments

Comments

@markgibbons25
Copy link

Toggling this class won't work because if you add or remove layers from the feature group it will just toggle the class on or off without taking into account whether the featuregroup actually has any layers.

@jacobtoye
Copy link
Member

Ah nice find, I calculate hasLayers but don't use it for toggling :| Will fix.

@joeybaker
Copy link
Contributor

This is kinda a big deal for us – any chance a quick update can be issued?

joeybaker added a commit to joeybaker/Leaflet.draw that referenced this issue Oct 12, 2013
Properly determines if it has layers, and then sets the button's disabled class.

fixes Leaflet#202
@jacobtoye
Copy link
Member

Thanks @joeybaker, I don't know what I was think writing that code :\

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants