Skip to content

Rewrite using Astro, TailwindCSS and DaisyUI#42

Merged
jacobwhall merged 10 commits intoMapRVA:trunkfrom
jacobwhall:trunk
Mar 27, 2026
Merged

Rewrite using Astro, TailwindCSS and DaisyUI#42
jacobwhall merged 10 commits intoMapRVA:trunkfrom
jacobwhall:trunk

Conversation

@jacobwhall
Copy link
Copy Markdown
Member

This is a full rewrite of the site. It builds a lot faster, and the codebase is much smaller.

Highlights:

  • Homepage has a map in the background!
  • Events are auto-populated from Google Calendar at build time
  • Astro makes building the site, both in development and prod, much easier
  • New /in-the-news/ page shows all of our shout-outs in news articles
  • Initial work on Sveltia CMS integration will allow us to edit site content in a more user-friendly way

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.

@jacobwhall jacobwhall requested review from dschep, jpole1, kfinity and mhpob and removed request for jpole1 March 21, 2026 21:15
@jacobwhall
Copy link
Copy Markdown
Member Author

I requested reviews from all MapRVA members who have contributed to the repository thus far.

Copy link
Copy Markdown
Contributor

@kfinity kfinity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 🥲

@jacobwhall
Copy link
Copy Markdown
Member Author

Thanks, @kfinity!

that great background map on the homepage is a little slow to load on mobile

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.

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

@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

Copy link
Copy Markdown
Contributor

@mhpob mhpob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me -- any changes I'd request have already been addressed.

Echo Kevin's suggestion re: map background.

@jacobwhall
Copy link
Copy Markdown
Member Author

I admit it's fun to have a high-res background map in widescreen...but I won't die on this hill.

Screenshot From 2026-03-22 17-55-14

@dschep
Copy link
Copy Markdown
Member

dschep commented Mar 25, 2026

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

@dschep
Copy link
Copy Markdown
Member

dschep commented Mar 25, 2026

https://maprva.github.io/ultra-maps/

@jacobwhall
Copy link
Copy Markdown
Member Author

ultra queries just don't seem to be hosted where they're expected

These should be hosted in a backwards-compatible way, e.g. /ultra-maps/surveillance.ultra. I also tested /img/surveillance-direction.png

https://maprva.github.io/ultra-maps/

Even better!

@jacobwhall jacobwhall merged commit d712d7d into MapRVA:trunk Mar 27, 2026
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.

4 participants