authorgravatar for 76637177+SunkenInTime@users.noreply.github.comDara Adedeji <76637177+SunkenInTime@users.noreply.github.com> 2026-10-11 17:35:03-04:00
committergravatar for noreply@github.comGitHub <noreply@github.com> 2026-10-11 18:35:03-03:00
logf43f6ddde06047d8d53da0abb355b41d703562ce
tree2f3b146598cb58735f24cb210710fc0d1b63c19b
parent1d270bec29d74ed31b69577f7f8aee5541457d21
signature Signed by PGP key B5690EEEBB952194

fix(web): even 4px inset around rows in the PR badge tooltip (#18352)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

1 files changed, 18 insertions(+), 14 deletions(-)

apps/web/src/components/ThreadStatusIndicators.tsx+18-14
...@@ -316,22 +316,26 @@ function PullRequestBadge({...@@ -316,22 +316,26 @@ function PullRequestBadge({
316 variant="glass"316 variant="glass"
317 className="pointer-events-auto w-80 max-w-[calc(100vw-2rem)] text-left whitespace-normal"317 className="pointer-events-auto w-80 max-w-[calc(100vw-2rem)] text-left whitespace-normal"
318 >318 >
319 {visibleThreadPullRequests(pullRequests).length > 0 ? (319 {/* The viewport pads 8px inline and 4px block; pull the rows out so their hover
320 <ThreadPullRequestsMiniList320 highlight sits 4px from every edge. */}
321 pullRequests={pullRequests}321 <div className="-mx-1">
322 onOpenPullRequest={onOpenPullRequest}322 {visibleThreadPullRequests(pullRequests).length > 0 ? (
323 />323 <ThreadPullRequestsMiniList
324 ) : number !== undefined && url !== undefined ? (324 pullRequests={pullRequests}
325 <ul className="flex flex-col gap-1">
326 <ThreadPullRequestMiniListItem
327 number={number}
328 url={url}
329 title={status?.tooltipTitle ?? presentation.label}
330 presentation={presentation}
331 onOpenPullRequest={onOpenPullRequest}325 onOpenPullRequest={onOpenPullRequest}
332 />326 />
333 </ul>327 ) : number !== undefined && url !== undefined ? (
334 ) : null}328 <ul className="flex flex-col gap-1">
329 <ThreadPullRequestMiniListItem
330 number={number}
331 url={url}
332 title={status?.tooltipTitle ?? presentation.label}
333 presentation={presentation}
334 onOpenPullRequest={onOpenPullRequest}
335 />
336 </ul>
337 ) : null}
338 </div>
335 </TooltipPopup>339 </TooltipPopup>
336 </Tooltip>340 </Tooltip>
337 );341 );