Feature/corrected coords consolidation - #940
Merged
AgreeDK merged 5 commits intoSep 29, 2026
Merged
Conversation
…he table's right-click menu and the map's right-click menu
10 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
related to #936 ping me if a rebranch will be needed after the other PR's
Corrected coordinates: consolidate entry points, add distance/bearing and clear button
Corrected coordinates can be set from the detail panel, the cache table menu, the map menu and the Edit Cache dialog. Each entry point had its own database-write code and refreshed the UI differently, which led to stale views and dialogs that looked different.
Changes
set_corrected_coords()indb/corrected_coords.pyis used by the detail panel, the table menu and the map menu._on_corrected_coords_changednow also refreshes the detail panel when it shows the affected cache. Previously it stayed stale after a table or map change.Tests
New unit and e2e tests cover the shared DB write, detail-panel refresh from the table and the map, the distance/bearing line and the clear button. The full unit and e2e suites pass.