From de39328f38e8a265973d60fd31d51627a044268d Mon Sep 17 00:00:00 2001 From: maria-rcks Date: Sat, 12 Sep 2026 03:02:34 +0000 Subject: [PATCH 1/2] fix(web): compact pr description reactions --- .../web/src/components/pullRequest/PullRequestReactions.tsx | 6 +++++- .../src/components/pullRequest/PullRequestSummaryTab.tsx | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/apps/web/src/components/pullRequest/PullRequestReactions.tsx b/apps/web/src/components/pullRequest/PullRequestReactions.tsx index 2ef90b09a55c..a5e15ff2d40d 100644 --- a/apps/web/src/components/pullRequest/PullRequestReactions.tsx +++ b/apps/web/src/components/pullRequest/PullRequestReactions.tsx @@ -51,6 +51,7 @@ export function PullRequestReactionBar({ reference, onRefresh, className, + compact = false, }: { readonly reactions: ReadonlyArray; readonly canReact: boolean; @@ -60,6 +61,7 @@ export function PullRequestReactionBar({ readonly reference: PullRequestRef; readonly onRefresh: () => void; readonly className?: string | undefined; + readonly compact?: boolean; }) { const [pickerOpen, setPickerOpen] = useState(false); const [pending, setPending] = useState<{ @@ -110,6 +112,7 @@ export function PullRequestReactionBar({ disabled={!canReact} className={cn( PILL_CLASS, + compact && "h-5 px-1.5", reaction.viewerHasReacted ? "border-primary/60 bg-primary/10 text-foreground" : "border-border/70 bg-muted/40 text-muted-foreground", @@ -136,6 +139,7 @@ export function PullRequestReactionBar({ className={cn( PILL_CLASS, "border-border/70 px-1.5 text-muted-foreground hover:border-primary/60 hover:text-foreground", + compact && "h-5 px-1", shown.length === 0 && !pickerOpen && "opacity-0 group-focus-within:opacity-100 group-hover:opacity-100 focus-visible:opacity-100", @@ -143,7 +147,7 @@ export function PullRequestReactionBar({ /> } > - +
diff --git a/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx b/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx index fa6256394913..7d023d3d0a48 100644 --- a/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx +++ b/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx @@ -758,7 +758,7 @@ export function PullRequestSummaryTab({
)} Date: Sat, 12 Sep 2026 03:13:53 +0000 Subject: [PATCH 2/2] fix(web): remove description reaction row --- .../src/components/pullRequest/PullRequestReactions.tsx | 6 +----- .../src/components/pullRequest/PullRequestSummaryTab.tsx | 8 -------- 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/apps/web/src/components/pullRequest/PullRequestReactions.tsx b/apps/web/src/components/pullRequest/PullRequestReactions.tsx index a5e15ff2d40d..2ef90b09a55c 100644 --- a/apps/web/src/components/pullRequest/PullRequestReactions.tsx +++ b/apps/web/src/components/pullRequest/PullRequestReactions.tsx @@ -51,7 +51,6 @@ export function PullRequestReactionBar({ reference, onRefresh, className, - compact = false, }: { readonly reactions: ReadonlyArray; readonly canReact: boolean; @@ -61,7 +60,6 @@ export function PullRequestReactionBar({ readonly reference: PullRequestRef; readonly onRefresh: () => void; readonly className?: string | undefined; - readonly compact?: boolean; }) { const [pickerOpen, setPickerOpen] = useState(false); const [pending, setPending] = useState<{ @@ -112,7 +110,6 @@ export function PullRequestReactionBar({ disabled={!canReact} className={cn( PILL_CLASS, - compact && "h-5 px-1.5", reaction.viewerHasReacted ? "border-primary/60 bg-primary/10 text-foreground" : "border-border/70 bg-muted/40 text-muted-foreground", @@ -139,7 +136,6 @@ export function PullRequestReactionBar({ className={cn( PILL_CLASS, "border-border/70 px-1.5 text-muted-foreground hover:border-primary/60 hover:text-foreground", - compact && "h-5 px-1", shown.length === 0 && !pickerOpen && "opacity-0 group-focus-within:opacity-100 group-hover:opacity-100 focus-visible:opacity-100", @@ -147,7 +143,7 @@ export function PullRequestReactionBar({ /> } > - +
diff --git a/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx b/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx index 7d023d3d0a48..2c8bb76aafd3 100644 --- a/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx +++ b/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx @@ -757,14 +757,6 @@ export function PullRequestSummaryTab({ ) : null}
)} -