Skip to content

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

Open
dudina-ma wants to merge 2 commits into
masterfrom
fix/ui-project-breadcrumb-links
Open

[frontend] open project from task crumbs in a new tab#42
dudina-ma wants to merge 2 commits 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 hash-based links instead of clickable buttons, improving navigation reliability.
    • Ensured saved task-filter snapshot lookups run without reactive tracking to prevent unnecessary updates while keeping pagination and filtering behavior the same.

@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: dfb46247-89d3-4fc3-8e6f-80f91b154783

📥 Commits

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

📒 Files selected for processing (1)
  • frontend/src/lib/pages/project-tasks.svelte

📝 Walkthrough

Walkthrough

The frontend updates recent-project tracking and task-filter snapshot reads to avoid Svelte dependency tracking, and changes task-detail project breadcrumbs to hash-based anchor links.

Changes

Frontend navigation behavior

Layer / File(s) Summary
Non-reactive state reads
frontend/src/App.svelte, frontend/src/lib/pages/project-tasks.svelte
Wraps recent-project tracking and route-key snapshot lookup in untrack(...).
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#37: Introduces the snapshot and route-key persistence logic used by the updated non-reactive task-filter lookup.

Suggested labels: deployed, ready

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 reasonably reflects the main user-facing update.
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
@github-actions github-actions Bot removed ready PR is ready to merge deployed PR is deployed to staging labels Jul 22, 2026
@coderabbitai coderabbitai Bot added deployed PR is deployed to staging ready PR is ready to merge labels Jul 22, 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