Skip to content

fix(web): show pointer cursor on pull request interactive controls - #7629

Open
mdshzb04 wants to merge 1 commit into
pingdotgg:mainfrom
mdshzb04:fix/pr-screen-pointer-cursors
Open

fix(web): show pointer cursor on pull request interactive controls#7629
mdshzb04 wants to merge 1 commit into
pingdotgg:mainfrom
mdshzb04:fix/pr-screen-pointer-cursors

Conversation

@mdshzb04

@mdshzb04 mdshzb04 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • vp test run apps/web/src/components/pullRequest/PullRequestRow.test.tsx
  • Open Pull Requests → hover list rows, PR number, timeline commits, checks, reactions, reviewer picker

Note

Low Risk
Tailwind class-only UX polish on pull request UI with no behavior, data, or auth changes.

Overview
Adds cursor-pointer to native <button> elements across the pull request list and detail flow so hover states read as clickable. Shared Button / Toggle already set the pointer; this targets raw buttons that were still using the default arrow cursor.

Coverage includes list rows, PR number links, timeline commit rows, check rows (pointer only when a URL exists; cursor-default when not), reactions and the emoji picker, reviewer picker rows (disabled:cursor-not-allowed while a request is in flight), review thread expand/collapse, and check Details links in the popover.

Adds PullRequestRow.test.tsx to assert selectable rows render with cursor-pointer in static markup.

Reviewed by Cursor Bugbot for commit bac1e2c. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Add cursor-pointer to interactive pull request controls in web UI

  • Adds the cursor-pointer utility class to clickable elements across multiple pull request components: the Details button in ChecksBody, the host-open button in PullRequestDetailPanel, reaction buttons and emoji options in PullRequestReactionBar, thread header toggles in ReviewThreadCard, candidate buttons in PullRequestReviewerPicker, the root button in PullRequestRow, and commit event buttons in CommitEvent
  • In PullRequestSummaryTab, per-check rows now use cursor-pointer when a URL is present and cursor-default otherwise
  • Adds a test in PullRequestRow.test.tsx asserting selectable rows render cursor-pointer in static markup

Macroscope summarized bac1e2c.

Raw buttons in the pull request list and detail flow did not set
cursor-pointer, so clickable rows and controls kept the default arrow
on macOS desktop. Closes pingdotgg#7606.
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

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: f2d2653b-5890-47a9-be7f-7bb77bd4586a

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

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

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Aug 20, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at bac1e2c

Macroscope's review found this PR approvable — Pure CSS styling changes adding cursor-pointer to interactive elements. No logic or behavioral changes - only visual cursor feedback improvements.

You can add or adjust custom eligibility rules. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant