Skip to content

Refactor navigation pins and interaction surfaces#17

Merged
dovvnloading merged 1 commit into
mainfrom
codex/navigation-pins-refactor
Jul 17, 2026
Merged

Refactor navigation pins and interaction surfaces#17
dovvnloading merged 1 commit into
mainfrom
codex/navigation-pins-refactor

Conversation

@dovvnloading

Copy link
Copy Markdown
Owner

Summary

  • refactor navigation pins around a validated, Qt-free record store and controller
  • replace the legacy pin overlay/list rebuild path with a model-driven in-window panel and shared editor
  • harden pin graphics geometry, drag persistence, context actions, stable IDs, ordering, and database migration
  • keep navigation pins out of Fit All by default and route creation/edit/delete/focus through one scene boundary
  • preserve the verified neutral grayscale UI and composer updates in the same local release snapshot

Root cause

Navigation pins previously had multiple mutable sources of truth, a window-coupled overlay, duplicated editor code, prototype-level graphics geometry, and persistence without stable identifiers or deterministic ordering. That combination caused stale panel state, repaint trails, fragile deletion paths, and migration risk.

Validation

  • python -m pytest -q --basetemp .pytest-tmp — 475 passed
  • Qt runtime smoke test for the pin panel, store-backed model, and literal filtering passed
  • targeted Python compilation passed
  • git diff --check passed

The local-only doc/ proposal remains intentionally excluded from this public repository.

@dovvnloading
dovvnloading merged commit 889c3b3 into main Jul 17, 2026
5 of 7 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