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

Prepend default __btnIcon with currentScript url #214

Merged
merged 1 commit into from
Sep 21, 2022
Merged

Prepend default __btnIcon with currentScript url #214

merged 1 commit into from
Sep 21, 2022

Conversation

Raruto
Copy link
Owner

@Raruto Raruto commented Sep 20, 2022

Close: #209

@Raruto
Copy link
Owner Author

Raruto commented Sep 20, 2022

@hupe13 can you confirm that this pull request does not change the current behavior? (the url of the downloaded svg file should be relative to the current location of the leaflet-elevation.js, that is: your local web server or alternatively unpkg.com)

@hupe13
Copy link
Contributor

hupe13 commented Sep 21, 2022

I changed the line in my WordPress Plugin and logged the baseURL. It is correct and there is no error when calling a test page.

@Raruto Raruto merged commit dea454a into master Sep 21, 2022
@Raruto Raruto deleted the btn-icon branch September 21, 2022 16:12
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

Successfully merging this pull request may close these issues.

No more elevation icon
2 participants