Skip to content

Copy coordinates on right click#732

Merged
louis-e merged 2 commits intomainfrom
copy-coords
Feb 1, 2026
Merged

Copy coordinates on right click#732
louis-e merged 2 commits intomainfrom
copy-coords

Conversation

@louis-e
Copy link
Owner

@louis-e louis-e commented Feb 1, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 1, 2026 00:05
@github-actions
Copy link

github-actions bot commented Feb 1, 2026

⏱️ Benchmark run finished in 0m 40s
🧠 Peak memory usage: 959 MB

📈 Compared against baseline: 30s
🧮 Delta: 10s
🔢 Commit: 674a2d9

⚠️ This PR worsens generation time.

📅 Last benchmark: 2026-02-01 00:09:20 UTC

You can retrigger the benchmark by commenting retrigger-benchmark.

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 adds functionality to copy Minecraft coordinates via a right-click context menu on the map interface. The feature calculates Minecraft world coordinates from geographic coordinates and formats them as a teleport command for easy copying.

Changes:

  • Added JavaScript implementation for context menu display, coordinate calculation, and clipboard operations
  • Added CSS styling for the context menu with hover effects and proper positioning
  • Extended Rust backend to extract and expose Minecraft coordinate bounds from world metadata

Reviewed changes

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

File Description
src/gui/js/bbox.js Implements context menu creation, coordinate conversion logic, clipboard API with fallback, and event handling for right-click interactions
src/gui/css/bbox.css Defines styles for the context menu including positioning, hover states, and unused animation keyframes
src/gui.rs Extends WorldMapData structure with Minecraft coordinate bounds (min/max X and Z) extracted from metadata

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

@louis-e louis-e merged commit d4af5ce into main Feb 1, 2026
2 checks passed
@louis-e louis-e deleted the copy-coords branch February 1, 2026 00:09
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.

2 participants