-
Notifications
You must be signed in to change notification settings - Fork 2
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
Filter fixed in osm-verkehrswende-repo #1
Comments
Thanks for the fixes, I already saw the commits and was thinking about applying them once I have some time. And thanks again for your great work! |
I just went through all of the commits and indeed applying them manually wasn't that hard (thanks to typescript). I just noticed two things: There are a lot of JS errors like And because of the id-change in the legend, the colors in the mapinspector are broken, but that's a simple fix you should be able to directly reuse: 13ed2d2 |
Perfect, thanks!
Yeah, this is super annoying and I don't know why it happens. I have asked about it in the maplibre Slack but no answer. And I started looking into it in maplibre/maplibre-style-spec#555 but did not get very far. TBH I don't even understand why I cannot find any issues on it, because I think I am doing everything like I should … so it cannot be that I am the only one having those issues. 🤷 — I am very open to suggestions :).
Thanks, great catch, did the same in osmberlin/www.osm-verkehrswende.org@b1d4a29 |
FYI, I fixed and improved the filter system in https://github.com/osmberlin/www.osm-verkehrswende.org/commits/main/
I did not look into how we might keep the codebase of this stand alone version in sync with the project website. Might be easiest to copy paste the components. However, we should extract the parts that need to be configured per page. Likely the pmtiles URL mainly?
The text was updated successfully, but these errors were encountered: