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

ES module support? #313

Closed
DavidMStraub opened this issue Sep 25, 2022 · 1 comment · Fixed by #357
Closed

ES module support? #313

DavidMStraub opened this issue Sep 25, 2022 · 1 comment · Fixed by #357

Comments

@DavidMStraub
Copy link

Hi, thanks for this great plugin. I was wondering if you plan to provide an ES module version of it. The release announcements of Leaflet 1.9.0 advocate the use of ES modules and it is planned to even remove the L global in the future. It would be great if I could just do

import leaflet
import leaflet.locatecontrol

in the future 🙂

@domoritz
Copy link
Owner

I'd be happy to accept a pull request for an esm version. The difficulty is that we'd need to set up some compilation process for the non esm version. The benefits are also not as large as with other libraries since the code is so small.

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