Skip to content

[Application] Left navigation tree context menu actions#204

Merged
learncold merged 2 commits into
mainfrom
codex/nav-tree-context-menu
May 9, 2026
Merged

[Application] Left navigation tree context menu actions#204
learncold merged 2 commits into
mainfrom
codex/nav-tree-context-menu

Conversation

@95x8x9
Copy link
Copy Markdown
Collaborator

@95x8x9 95x8x9 commented May 8, 2026

Summary

  • Adds right-click context menus to left navigation trees (Layout/Crowd/Events) to allow Delete, plus Settings where applicable.
  • Fixes a crash caused by parenting a stack-allocated QMenu to the tree while the navigation panel is being replaced.

Related Issue

  • None (application-only PR)

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • Not run (reason below)

Risks / Follow-up

  • Operational event Settings currently edits the name/trigger/target summaries via a simple dialog; future work may replace this with a richer inspector UI.

@95x8x9 95x8x9 marked this pull request as ready for review May 8, 2026 13:42
@95x8x9 95x8x9 requested a review from learncold as a code owner May 8, 2026 13:42
@learncold learncold force-pushed the codex/nav-tree-context-menu branch from c0f8739 to 88bb678 Compare May 9, 2026 09:29
@learncold learncold merged commit 0687e70 into main May 9, 2026
2 checks passed
@learncold learncold deleted the codex/nav-tree-context-menu branch May 9, 2026 09:30
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