Skip to content

Conversation

@arnesetzer
Copy link
Contributor

@arnesetzer arnesetzer commented Jul 28, 2025

👋 I did some awesome work for the Pelias project and would love for everyone to have a look at it and provide feedback.


Here's the reason for this change 🚀

As discussed in #373 here is updated version with https://github.com/geocodeearth/autocomplete-element instead of nextzen-leaflet. Due to laziness the url (localhost:4000) is hardcoded in the bundle.js. We could link the api in the docker-compose.yml which would allow us to use the container name instead of ip but wont solve the problem with the port. But since this is only nice2have I don't see it as this big of a problem.


Here's what actually got changed 👏

  • Update the leaflet packages to the latest version
  • Fix the stamen tile url
  • replace nextzen-leaflet witjk autocomplete-element

Here's how others can test the changes 👀

Fire up any project with existing data, go to "localhost:3000" and search something.

Closes #373

@arnesetzer arnesetzer changed the title Feat: Update preview feat: Update preview Aug 6, 2025
@orangejulius
Copy link
Member

I forgot to reply to this earlier, but it's very nice. Maybe we wait until after #386 is merged so that this diff is much smaller and simpler?

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.

State of the common/preview directory

2 participants