Skip to content

fix(conversations): Open conversation in new tab on cmd/ctrl+click - #121487

Merged
ArthurKnaus merged 2 commits into
masterfrom
arthurknaus/tet-2803-conversations-do-not-open-in-a-new-tab
Aug 7, 2026
Merged

fix(conversations): Open conversation in new tab on cmd/ctrl+click#121487
ArthurKnaus merged 2 commits into
masterfrom
arthurknaus/tet-2803-conversations-do-not-open-in-a-new-tab

Conversation

@ArthurKnaus

@ArthurKnaus ArthurKnaus commented Aug 6, 2026

Copy link
Copy Markdown
Member

Conversation rows navigate via a JS onClick handler rather than a real link, so Cmd/Ctrl+click had no URL to act on and just navigated in place instead of opening a new tab.

Reads the click event's modifier keys and uses window.open instead of navigate() when Cmd/Ctrl (new tab) or Shift (new window) is held, matching native link behavior.

Refs TET-2803

Conversation rows navigated via a JS onClick handler, so Cmd/Ctrl+click
had no URL to act on and just navigated in place instead of opening a new
tab. Read the click event's modifier keys and use window.open instead of
navigate() when Cmd/Ctrl (new tab) or Shift (new window) is held, matching
native link behavior.

Refs TET-2803
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Aug 6, 2026

Copy link
Copy Markdown

TET-2803

@ArthurKnaus
ArthurKnaus marked this pull request as ready for review August 6, 2026 21:36
@ArthurKnaus
ArthurKnaus requested a review from a team as a code owner August 6, 2026 21:36
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 7, 2026
@ArthurKnaus
ArthurKnaus merged commit e5f2b73 into master Aug 7, 2026
70 checks passed
@ArthurKnaus
ArthurKnaus deleted the arthurknaus/tet-2803-conversations-do-not-open-in-a-new-tab branch August 7, 2026 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants