fix(web): show pointer cursor on pull request interactive controls - #7629
fix(web): show pointer cursor on pull request interactive controls#7629mdshzb04 wants to merge 1 commit into
Conversation
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.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — Pure CSS styling changes adding You can add or adjust custom eligibility rules. Learn more. |
Summary
cursor-pointerto raw<button>controls in the pull request list and detail flow ([Bug]: Pull request screens are missing pointer cursors on clickable elements #7606)Toggle, which includescursor-pointeron mainTest plan
vp test run apps/web/src/components/pullRequest/PullRequestRow.test.tsxNote
Low Risk
Tailwind class-only UX polish on pull request UI with no behavior, data, or auth changes.
Overview
Adds
cursor-pointerto native<button>elements across the pull request list and detail flow so hover states read as clickable. SharedButton/Togglealready 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-defaultwhen not), reactions and the emoji picker, reviewer picker rows (disabled:cursor-not-allowedwhile a request is in flight), review thread expand/collapse, and check Details links in the popover.Adds
PullRequestRow.test.tsxto assert selectable rows render withcursor-pointerin 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-pointerto interactive pull request controls in web UIcursor-pointerutility class to clickable elements across multiple pull request components: the Details button inChecksBody, the host-open button inPullRequestDetailPanel, reaction buttons and emoji options inPullRequestReactionBar, thread header toggles inReviewThreadCard, candidate buttons inPullRequestReviewerPicker, the root button inPullRequestRow, and commit event buttons inCommitEventPullRequestSummaryTab, per-check rows now usecursor-pointerwhen a URL is present andcursor-defaultotherwisecursor-pointerin static markupMacroscope summarized bac1e2c.