Skip to content

Commit 61ea6ed

Browse files
committed
fix(ui): reduce spacing between notification messages
1 parent 31e452e commit 61ea6ed

2 files changed

Lines changed: 4 additions & 8 deletions

File tree

ui/src/components/notifications/NotificationCard.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ function CardFrame({
5353
return (
5454
<Card
5555
className={cn(
56-
"group transition-colors hover:bg-accent/50 border-l-4",
56+
"group transition-colors hover:bg-accent/50 border-l-4 py-3 gap-2",
5757
borderColor,
5858
isUnread && "bg-accent/20",
5959
onClick && "cursor-pointer",

ui/src/components/notifications/NotificationList.tsx

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import Hash from "lucide-react/dist/esm/icons/hash";
33
import Inbox from "lucide-react/dist/esm/icons/inbox";
44
import { memo, useCallback, useMemo, useRef, useState } from "react";
55
import { Button } from "@/components/ui/button";
6-
import { Separator } from "@/components/ui/separator";
76
import {
87
Tooltip,
98
TooltipContent,
@@ -149,8 +148,8 @@ export const NotificationList = memo(function NotificationList({
149148
ref={scrollContainerRef}
150149
className="flex-1 overflow-y-auto min-h-0"
151150
>
152-
<div className="p-4 space-y-3">
153-
{notifications.map((notification, index) => (
151+
<div className="p-3 space-y-1.5">
152+
{notifications.map((notification) => (
154153
<div
155154
key={notification.id}
156155
className={
@@ -160,7 +159,7 @@ export const NotificationList = memo(function NotificationList({
160159
}
161160
style={{
162161
contentVisibility: "auto",
163-
containIntrinsicSize: "auto 120px",
162+
containIntrinsicSize: "auto 80px",
164163
}}
165164
>
166165
<NotificationCard
@@ -174,9 +173,6 @@ export const NotificationList = memo(function NotificationList({
174173
isExpanded={!compactView || notification.isExpanded}
175174
onExpandedChange={handleExpandedChange}
176175
/>
177-
{index < notifications.length - 1 && (
178-
<Separator className="my-3 opacity-0" />
179-
)}
180176
</div>
181177
))}
182178
</div>

0 commit comments

Comments
 (0)