Rewrite using Astro, TailwindCSS and DaisyUI#42
Conversation
|
I requested reviews from all MapRVA members who have contributed to the repository thus far. |
kfinity
left a comment
There was a problem hiding this comment.
I love it - I think it looks great. I haven't tried building it locally yet. I tested most of the pages and they seem solid. The only feedback I can think of is super minor:
-
that great background map on the homepage is a little slow to load on mobile, compared to the rest of the page. Idk if it's a static image that's easy to scale down or something.
-
maybe we should switch the embedded ultra maps on the individual project pages to use the MapRVA overpass server, they're giving me "failed to fetch" errors 🥲
|
Thanks, @kfinity!
Yeah, right now its an entire MapLibre embed. Which I admit is a bit silly. I am open to making that a static image asset instead.
@dschep can confirm this but I think it's just a quirk of Ultra's CORS restrictions. I've had similar issues in the past in development environments. I expect that these embeds will work as expected once deployed to maprva.org |
|
yeah, ultra queries just don't seem to be hosted where they're expected. But.. I do agree with the idea of just having a separate repo for queries. I'll see if I can whip that up |
These should be hosted in a backwards-compatible way, e.g. Even better! |

This is a full rewrite of the site. It builds a lot faster, and the codebase is much smaller.
Highlights:
You can see how the site looks here: https://jacobwhall.github.io/maprva.org/
All feedback is welcome. I hope this is a positive step forward for the MapRVA website, and I think it'll make it easier to add more features and content in the future.