Merge pull request #4692 from Infisical/align-notification-icon

feat(notifications): UI fix for tailwind upgrade
This commit is contained in:
Andre
2025-10-20 12:16:06 -04:00
committed by GitHub
@@ -16,15 +16,15 @@ export const Notification = ({ notification, onDelete }: Props) => {
return ( return (
<div <div
className={twMerge( className={twMerge(
"group relative flex cursor-pointer items-start border-b border-mineshaft-600 p-3 transition-colors", "group relative flex cursor-pointer items-start border-b border-mineshaft-600 p-2 transition-colors",
notification.link ? "hover:bg-mineshaft-700" : "cursor-default", notification.link ? "hover:bg-mineshaft-700" : "cursor-default",
!notification.isRead && "bg-mineshaft-800" !notification.isRead && "bg-mineshaft-800"
)} )}
> >
<div className="flex w-full min-w-0 flex-col"> <div className="flex w-full min-w-0 flex-col p-1">
<div className="flex gap-2"> <div className="flex gap-2">
{!notification.isRead && ( {!notification.isRead && (
<FontAwesomeIcon icon={faCircle} className="mt-1.5 size-2 text-yellow-400" /> <FontAwesomeIcon icon={faCircle} className="mt-0.5 size-2 text-yellow-400" />
)} )}
<Tooltip <Tooltip
content={<Markdown>{notification.title}</Markdown>} content={<Markdown>{notification.title}</Markdown>}
@@ -45,7 +45,7 @@ export const Notification = ({ notification, onDelete }: Props) => {
</span> </span>
)} )}
</div> </div>
<div className="flex w-0 shrink-0 justify-end opacity-0 transition-all group-hover:w-[24px] group-hover:opacity-100"> <div className="mt-0.5 flex w-0 shrink-0 justify-end opacity-0 transition-all group-hover:w-[24px] group-hover:opacity-100">
<IconButton <IconButton
ariaLabel="delete" ariaLabel="delete"
variant="plain" variant="plain"