Skip to content

Allow navigation to and between activities in the heatmap#2

Merged
vbakke merged 3 commits intov4-devfrom
v4-feat/dep-nav-in-heatmap
Oct 11, 2025
Merged

Allow navigation to and between activities in the heatmap#2
vbakke merged 3 commits intov4-devfrom
v4-feat/dep-nav-in-heatmap

Conversation

@vbakke
Copy link
Copy Markdown
Owner

@vbakke vbakke commented Oct 11, 2025

By adding the '#{uuid}' in the url, one can now link to a specific activity in the heatmap, and also navigate between the activities using the dependency tree

@vbakke vbakke requested a review from Copilot October 11, 2025 08:07
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Enables deep-linking to specific heatmap activities via URL fragments (#uuid) and supports navigation between activities through the dependency graph.

  • Read and react to URL fragments to open the corresponding activity.
  • Update URL fragment when opening/closing the activity overlay; add dependency-graph click handler to navigate to activities.
  • Change activity detail opening to use UUIDs instead of names.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
src/assets/Markdown Files/TODO-v4.md Moves the heatmap deep-link/navigation item from TODO to Done.
src/app/pages/circular-heatmap/circular-heatmap.component.ts Adds fragment handling (ActivatedRoute), URL updates (Router/Location), navigation helpers, and switches activity detail opening to UUID.
src/app/pages/circular-heatmap/circular-heatmap.component.html Wires dependency graph click event and passes activity UUID to detail opener.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/app/pages/circular-heatmap/circular-heatmap.component.ts Outdated
Comment thread src/app/pages/circular-heatmap/circular-heatmap.component.ts Outdated
Comment thread src/app/pages/circular-heatmap/circular-heatmap.component.ts
Comment thread src/app/pages/circular-heatmap/circular-heatmap.component.ts
@vbakke vbakke merged commit bac6070 into v4-dev Oct 11, 2025
4 checks passed
@vbakke vbakke deleted the v4-feat/dep-nav-in-heatmap branch October 20, 2025 19:42
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