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 prevent unintended reactive updates.
    • Updated task detail breadcrumbs to use direct hash-based links for more reliable navigation.
    • Added breadcrumb links to the main Projects page and to the current project.

@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: 363e17e3-aca5-4cbd-af1c-b6f9ebc0392c

📥 Commits

Reviewing files that changed from the base of the PR and between e382d53 and 905fe45.

📒 Files selected for processing (2)
  • frontend/src/App.svelte
  • frontend/src/lib/pages/task-detail.svelte
🚧 Files skipped from review as they are similar to previous changes (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

Possibly related PRs

  • bit-issues/backend#38: Modifies the same recent-project tracking and hashchange logic in frontend/src/App.svelte.

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 accurately reflects the main change: task breadcrumb project links were converted to anchor links so they can open in a new tab.
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

github-actions Bot commented Jul 13, 2026

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

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 7 days with no activity.

@github-actions github-actions Bot added the stale label Jul 21, 2026
@capcom6 capcom6 removed the stale label Jul 21, 2026
@capcom6
capcom6 force-pushed the fix/ui-project-breadcrumb-links branch from e382d53 to 905fe45 Compare July 21, 2026 22:24
@capcom6 capcom6 added ready PR is ready to merge deployed PR is deployed to staging labels Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployed PR is deployed to staging ready PR is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants