Show selected grid cell on the map and polish map UI.#37
Conversation
Draw the snapped cell footprint with dashed guide lines, fly to the cell on click, and align the readout, nav, and control panel styling. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Code Review
This pull request transitions the map base styles to OpenFreeMap, introduces custom dark map style patching to improve label legibility, adds animated view state transitions when focusing on selected grid cells, and refactors the brand mark to use CSS masks. Feedback focuses on improving the robustness of the dark map style loading and fallback mechanism: specifically, allowing the darkMapStyle state to accept a string fallback URL, handling fetch failures by setting the fallback URL and clearing the cached promise to allow retries, and avoiding redundant label brightening loops when the style is already pre-patched.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
🚀 Vercel preview deployed Preview URL: https://earth-prints-gldsj4rau-anastasiia-s-projects10.vercel.app |
|
Looks good. And we should keep the pixel selected, however in #25 I meant to write that we should also show all the pixels that were cached, the Edit: The |
lazarusA
left a comment
There was a problem hiding this comment.
After looking at the performance and how smooth the clicking is I think we don't need to draw the corresponding chunk size grid fetched. So, we can leave that for later, and a different issue should be opened, plus, it should be optional, namely, the user should be able to show/hide the extent.
Summary
Review plan