Why does https://maplibre.org/maputnik fetch http://localhost:8000/styles ?
zstadler opened this issue · 7 comments
To reproduce:
- Open the browser's development console
- Load https://maplibre.org/maputnik
- Look for localhost:8000 in the network tab
Seems like this is related to the desktop communication.
See here:
Line 22 in 1da65f2
There's a mod of operation where the style is stored in a "backend" and not it local storage.
Should this be converted to a discussion or is there an issue you have encountered that is problematic?
There is no functional issue, but it is strange. I didn't expect to see any desktop-related functionality in the web site.
Was there a discussion about on-boarding the desktop?
There was a discussion on desktop to some extent here:
maplibre/maplibre#352
Currently, desktop is created as part of the CI of the web, I'm guessing that their functionality is not "separated" enough.
A build flag is what I would expect in this kind of scenario, it might be that I overlooked it when migrating to vite.
Should a new discussion on Maputnik Desktop be done in this repository, or in the MapLibre organization level?
Depending on what you would like to achieve.
I personally don't think anything else is needed in term of migrating a repository, as I see the desktop a "nice to have" king of a feature.
Having said that, if someone is willing to maintain it, s.he could open an on boarding issue and explain why this part should be in maplibre.
There's also another option to move this part of the code into this repo, but again, it will require someone who understands the code and would like to continue maintaining it.
Maintainence and maintainer are the key word here I guess.
I think desktop should be part of this repo, e.g. /desktop
, but only if someone can create a pr that:
- moves all the code with history (I can help with that)
- modifies ci to automate build and test
- updates the docs
- updates ci release process to auto publish the binary
I feel that in addition, the Maputnik Desktop also requires some UX changes.
For example when "watching" a local file and then opening a different file from the app, the first file is overwritten with the contents of the second file. Luckily for me, I had the first file also opened in an editor, so I was able to recover.
There is another option for the future of Desktop:
- Keep desktop as a separate repository
- Move all desktop-specific code to the desktop repository
- Have the maputnik CI/CD build, test and publish https://maplibre.org/maputnik
- Have the desktop CI/CD build, test and publish the desktop app, using code from the maputnik repository
I agree with @nyurik that any future support of desktop requires someone to prepare a PR.