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

Leaflet: update van voorbeeld #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Erikvv
Copy link

@Erikvv Erikvv commented Jul 14, 2023

Het leaflet voorbeeld werkte niet meer. Om het werkend te krijgen is het volgende gewijzigd:

  • CDN van de JS en CSS aangepast
  • Versie van leaflet aangepast naar de laatste
  • Domein van de WFS service aangepast

Verder zijn de volgende best practices toegepast

  • JQuery vervangen door Fetch API
  • CSS absolute postionering vervangen door Flexbox
  • JavaScrip var verangen door const

Ik heb ook het coordinatensysteem gelijkgetrokken zodat het voorbeeld enkel EPSG:4326 bevat.

Lost op:

Het leaflet voorbeeld werkte niet meer. Om het werkend te krijgen is het
volgende gewijzigd:

* CDN van de JS en CSS aangepast
* Versie van leaflet aangepast naar de laatste
* Domein van de WFS service aangepast

Verder zijn de volgende best practices toegepast

* JQuery vervangen door Fetch API
* CSS absolute postionering vervangen door Flexbox
* JavaScrip var verangen door const

Ik heb ook het coordinatensysteem gelijkgetrokken zodat het voorbeeld
enkel EPSG:4326 bevat.

Lost op:
- Geonovum#8
- Geonovum#39
@ndkv
Copy link
Contributor

ndkv commented Jul 14, 2023

Dank voor deze fijne pull! 🙌🏻

@lvdbrink ik review het morgen/zondagavond.

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.

2 participants