Skip to content

Conversation

@IEvangelist
Copy link
Member

@IEvangelist IEvangelist commented Oct 31, 2025

Rewrite the AspireMap.astro to:

  • Remove leaflet dep.
  • Add two new SVGs.
  • with subtle flicker effect on highlighted dots.

After: dark mode

image

After: light mode

image

@IEvangelist IEvangelist changed the title yeah yeah yeahs: Refactor code structure for improved readability and… yeah yeah yeahs: maps Oct 31, 2025
@IEvangelist IEvangelist requested a review from Copilot October 31, 2025 12:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR replaces an interactive Leaflet map component with a static SVG-based world map visualization. The change simplifies the implementation by removing the external astro-leaflet dependency and replacing it with theme-aware SVG images showing community locations with animated dots.

Key Changes:

  • Removed interactive Leaflet map library and dependencies
  • Added static SVG map images for light and dark themes with animated location markers
  • Replaced dynamic map component with a theme-aware image component
  • Removed Vite optimization configuration for Leaflet

Reviewed Changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
AspireMap.astro Replaced Leaflet map component with ThemeImage component using static SVG maps
map-lightdots.svg New SVG map for light theme with animated location dots
map-darkdots.svg New SVG map for dark theme with animated location dots
package.json Removed astro-leaflet dependency
package-lock.json Removed astro-leaflet, leaflet, and @types/leaflet package references
astro.config.mjs Removed Vite optimization configuration for leaflet
Files not reviewed (1)
  • src/frontend/package-lock.json: Language not supported

@IEvangelist IEvangelist merged commit a12b35a into main Oct 31, 2025
1 check passed
IEvangelist added a commit that referenced this pull request Oct 31, 2025
@IEvangelist IEvangelist deleted the maps branch November 11, 2025 19:44
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.

3 participants