Keep comment thread open when drag-selecting its text

[AnnaXWang]

Problem

Opening a comment thread and then click-dragging to highlight the comment text (to copy it) closes the thread on mouse release. Selecting text to copy shouldn't be a close gesture — the thread should stay open until explicitly closed.

Cause

The comment card's onClick toggles its pinned/open state for any click outside a data-no-pin region. A click-drag-release over the comment body still emits a click, so selecting text fired the toggle and closed the pinned thread. (The comment body isn't marked data-no-pin, unlike the actions row/composer/reactions.)

Fix

Distinguish a real click from a drag before toggling, in the card handler:

• record the pointer-down position on onMouseDown;
• on onClick, skip the toggle if the pointer moved more than ~4px (a drag), or if text ended up selected within the card.

A plain click still opens/closes the thread; a drag-to-highlight now leaves it open, so Cmd+C works. Explicit close paths (✕, clicking the highlight, switching cards) are unchanged.

Testing

• npm test — 108 passing, no regressions.
• tsc --noEmit clean.
• Manual: open a thread, drag-select the body → thread stays open and text is selectable/copyable; a normal click still toggles it.

Comment-card interactions aren't unit-tested in this repo (client-only DOM behavior), so the interaction check above is manual.

---

Note

Low Risk
Small client-only interaction tweak in the comment card click handler; no API or auth changes.

Overview
Comment rail cards no longer treat a drag-to-select over the comment body as a pin/unpin click. The card records pointer position on mousedown and, on click, skips onPin() when the pointer moved more than ~4px—so copy-by-highlighting leaves the thread open while a normal click still toggles it.

data-no-pin regions (actions, composer, reactions) are unchanged; only the card-level click handler gains this drag guard.

^{Reviewed by Cursor Bugbot for commit 2535b4a. Bugbot is set up for automated code reviews on this repo. Configure here.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
justhtml	Ready	Preview, Comment	Jul 8, 2026 11:34pm

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4ea2d56. Configure here.}

[rgarcia]

reviewed — clean, well-scoped fix. distinguishing a click from a drag-release via pointer movement is the right approach, comments explain the why, and no new deps. lgtm.