Skip to content

fix(node-ui): restore context graph detail origins#599

Merged
Jurij89 merged 4 commits into
mainfrom
codex/cgux-m2-back-navigation
May 23, 2026
Merged

fix(node-ui): restore context graph detail origins#599
Jurij89 merged 4 commits into
mainfrom
codex/cgux-m2-back-navigation

Conversation

@Jurij89
Copy link
Copy Markdown
Contributor

@Jurij89 Jurij89 commented May 22, 2026

Summary

  • Capture the Context Graph route/tab/scroll origin before entity drill-in and restore it when the detail view closes.
  • Keep cross-subgraph entity links inside the detail route instead of silently switching the underlying subgraph page.
  • Control layer and subgraph tabs from ProjectView, with stable scroll keys on the actual scroll roots used by restoration.
  • Rename the detail back action to "Back to Context Graph" for M2 terminology alignment.

Related

Files changed

File What
packages/node-ui/src/ui/views/ProjectView.tsx Adds origin snapshot/restore handling, controlled layer/subgraph tab state, and detail close routing.
packages/node-ui/src/ui/views/project/components.tsx Exposes controlled tabs, moves scroll keys onto stable scroll containers, and updates detail back copy.
packages/node-ui/test/project-view-navigation.test.ts Covers origin restoration and cross-subgraph detail navigation behavior.
packages/node-ui/test/ka-detail-label.test.ts Covers the rendered detail back-label copy.

Test plan

  • pnpm --filter @origintrail-official/dkg-node-ui exec vitest run test/project-view-navigation.test.ts test/ka-detail-label.test.ts test/ui-compat.test.ts test/use-memory-entities-live-updates.test.ts test/use-memory-entities-partial.test.ts
  • pnpm --filter @origintrail-official/dkg-node-ui run build
  • pnpm --filter @origintrail-official/dkg-node-ui run build:ui
  • git diff --check
  • In-app browser boot smoke at http://localhost:5176/ui/ with no console errors or warnings.
  • Local PR review: 4 rounds, final result had no actionable comments.

Note: the local dev node had zero context graphs, so the full live drill-in/back walkthrough still needs a populated-node manual pass.

Comment thread packages/node-ui/src/ui/views/ProjectView.tsx Fixed
Comment thread packages/node-ui/src/ui/views/ProjectView.tsx
Comment thread packages/node-ui/src/ui/views/project/components.tsx Outdated
Comment thread packages/node-ui/src/ui/views/ProjectView.tsx
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex review produced 3 comment(s) but all targeted lines outside the diff and were dropped. Check the workflow logs for details.

@Jurij89 Jurij89 force-pushed the codex/cgux-m2-back-navigation branch from 719247b to 5525545 Compare May 23, 2026 17:21
Comment thread packages/node-ui/src/ui/views/ProjectView.tsx Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex review completed — no issues found.

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