Skip to content

feat: map centering, route fit, and UI fixes#86

Merged
cahaseler merged 2 commits intomainfrom
fix-map-centering
Mar 24, 2026
Merged

feat: map centering, route fit, and UI fixes#86
cahaseler merged 2 commits intomainfrom
fix-map-centering

Conversation

@cahaseler
Copy link
Copy Markdown
Contributor

Summary

Galaxy map now centers on the player's current system on load instead of the world origin, and shows a persistent cyan ring/glow marker on the current location. When a route is plotted, the map automatically pans and zooms to show the full route. Also fixes the market insight indicators (invisible 6px dots replaced with lightbulb icons) and the recurring event log entry overlap bug.

User-Facing Changes

  • Play client galaxy map now centers on your current system when opened, with a visible "you are here" marker
  • Plotting a route automatically zooms the map to show the full route path
  • Market analysis insights now show as visible lightbulb icons next to item names (previously invisible dots) with an insight count summary
  • Fixed event log entries overlapping when longer than 2 lines

- Center galaxy map on player's current system on load instead of world origin
- Add persistent cyan ring + glow marker on player's current system
- Fit map view to show full route when one is first plotted
- Replace invisible 6px insight dot with Lightbulb icon badge
- Show insight count status bar after market analysis completes
- Fix event log entry overlap (re-add flex-shrink: 0 lost in prior refactor)
Prevents the route-fit from being silently skipped if a route is
plotted before map data finishes loading.
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
www Ready Ready Preview, Comment Mar 24, 2026 3:27pm

Request Review

@cahaseler cahaseler merged commit d600442 into main Mar 24, 2026
3 checks passed
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.

1 participant