fix(web): show pointer cursor on PR rows and reactions

[BeeSting50]

Problem

Pull request list rows and reaction controls remain clickable while showing the default cursor. The filter and detail tabs now use shared Button and Toggle primitives on current main, so those already show the correct pointer cursor.

Fix

• show the pointer cursor on selectable pull request rows
• show the pointer cursor on enabled reaction pills
• show the pointer cursor on the add-reaction trigger and emoji picker options
• preserve the default cursor for disabled reaction pills
• add a focused regression test for selectable rows

Verification

• node_modules/.bin/vp test run apps/web/src/components/pullRequest/PullRequestRow.test.tsx apps/web/src/components/pullRequest/pullRequestReactions.logic.test.ts (14 tests pass)
• targeted formatting passes
• git diff --check passes
• web typecheck reaches two unrelated existing errors in electronPasskeys.test.ts (conditionalUI type mismatch)

Closes #7606

Built with GPT-5.6 Codex in the T3 Code harness.

---

Note

Low Risk
CSS-only cursor classes and a static HTML regression test; no behavior or data-path changes.

Overview
Extends pointer-cursor affordances on pull request UI so clickable controls no longer show the default arrow.

PullRequestRow row buttons get cursor-pointer so selectable list rows read as clickable. In PullRequestReactions, the same class is applied to existing reaction pills (when reacting is allowed), the “Add a reaction” trigger, and each emoji in the reaction picker.

A new PullRequestRow.test.tsx uses static markup to assert selectable rows include cursor-pointer.

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

Note

Add cursor-pointer to PullRequestRow and PullRequestReactionBar controls

Adds the cursor-pointer utility to the row button in PullRequestRow.tsx and to reaction pill buttons, the popover trigger, and emoji option buttons in PullRequestReactions.tsx. A test in PullRequestRow.test.tsx verifies the class is present. No logic or event handling changes.

^{Macroscope summarized eb57ab4.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0c291986-222a-4e1d-8162-5538db3813d4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[macroscopeapp]

One inconsistency: the condensed copy of the same tab strip in PullRequestDetailPanel.tsx still renders without a pointer cursor, so the tabs change cursor behavior depending on whether the header is condensed.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Approved at eb57ab4

Macroscope's review found this PR approvable — Adds cursor-pointer CSS styling to PR rows and reaction buttons - purely visual change with no runtime behavior impact. Includes a simple test to verify the styling.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 51e2738

Dismissing prior approval to re-evaluate eb57ab4