Skip to content

[frontend] open project from task crumbs in a new tab#42

Open
dudina-ma wants to merge 1 commit into
masterfrom
fix/ui-project-breadcrumb-links
Open

[frontend] open project from task crumbs in a new tab#42
dudina-ma wants to merge 1 commit into
masterfrom
fix/ui-project-breadcrumb-links

Conversation

@dudina-ma

@dudina-ma dudina-ma commented Jul 13, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Bug Fixes
    • Improved recent-project tracking behavior to avoid unintended reactive updates.
    • Updated task detail breadcrumbs to use direct links for more reliable navigation.
    • Added links to the Projects page and the current project from task details.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 136128cd-f624-4c14-b555-3a7f782e72d3

📥 Commits

Reviewing files that changed from the base of the PR and between 8d80806 and e382d53.

📒 Files selected for processing (2)
  • frontend/src/App.svelte
  • frontend/src/lib/pages/task-detail.svelte

📝 Walkthrough

Walkthrough

The frontend updates recent-project tracking to run outside Svelte dependency tracking and changes task-detail project breadcrumbs from button handlers to hash-based anchor links.

Changes

Frontend navigation behavior

Layer / File(s) Summary
Non-reactive recent tracking
frontend/src/App.svelte
Imports untrack and wraps the initial trackRecentFromPath call while preserving the existing effect and event listener setup.
Task breadcrumb links
frontend/src/lib/pages/task-detail.svelte
Replaces project breadcrumb buttons and navigate(...) calls with links to #/projects and the project-specific hash route.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: capcom6

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the breadcrumb link change, and "open in a new tab" is a plausible consequence of switching to anchor links.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot requested a review from capcom6 July 13, 2026 20:11
@github-actions

Copy link
Copy Markdown

🤖 Pull request artifacts

Platform File
🐳 Docker GitHub Container Registry
🍎 Darwin arm64 backend_Darwin_arm64.tar.gz
🍎 Darwin x86_64 backend_Darwin_x86_64.tar.gz
🐧 Linux arm64 backend_Linux_arm64.tar.gz
🐧 Linux i386 backend_Linux_i386.tar.gz
🐧 Linux x86_64 backend_Linux_x86_64.tar.gz
🪟 Windows arm64 backend_Windows_arm64.zip
🪟 Windows i386 backend_Windows_i386.zip
🪟 Windows x86_64 backend_Windows_x86_64.zip

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