From 3758941a59d85307562d5a6a49267bf974fbebc4 Mon Sep 17 00:00:00 2001
From: Adam Setch <adam.setch@outlook.com>
Date: Wed, 22 Jan 2025 15:36:35 -0500
Subject: [PATCH] refactor: notification footer

Signed-off-by: Adam Setch <adam.setch@outlook.com>
---
 .../notifications/NotificationFooter.tsx      |  5 +-
 .../AccountNotifications.test.tsx.snap        | 28 ++------
 .../NotificationFooter.test.tsx.snap          | 70 +++----------------
 .../NotificationRow.test.tsx.snap             | 42 ++---------
 4 files changed, 23 insertions(+), 122 deletions(-)

diff --git a/src/renderer/components/notifications/NotificationFooter.tsx b/src/renderer/components/notifications/NotificationFooter.tsx
index cfe54ddbc..445799075 100644
--- a/src/renderer/components/notifications/NotificationFooter.tsx
+++ b/src/renderer/components/notifications/NotificationFooter.tsx
@@ -53,8 +53,9 @@ export const NotificationFooter: FC<INotificationFooter> = ({
       )}
 
       <Stack direction="horizontal" gap="none">
-        <Text title={reason.description}>{reason.title}</Text>
-        <Text className="px-1">&bull;</Text>
+        <Text title={reason.description} className="pr-1">
+          {reason.title}
+        </Text>
         <RelativeTime datetime={notification.updated_at} />
       </Stack>
 
diff --git a/src/renderer/components/notifications/__snapshots__/AccountNotifications.test.tsx.snap b/src/renderer/components/notifications/__snapshots__/AccountNotifications.test.tsx.snap
index 13710501e..57c103815 100644
--- a/src/renderer/components/notifications/__snapshots__/AccountNotifications.test.tsx.snap
+++ b/src/renderer/components/notifications/__snapshots__/AccountNotifications.test.tsx.snap
@@ -881,16 +881,11 @@ exports[`renderer/components/notifications/AccountNotifications.tsx should rende
               data-wrap="nowrap"
             >
               <span
-                class="Text-sc-17v1xeu-0 hmAFGJ"
+                class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
                 title="You're watching the repository."
               >
                 Updated
               </span>
-              <span
-                class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-              >
-                •
-              </span>
               <relative-time
                 class="sc-KXCwU"
                 datetime="2017-05-20T17:51:57.000Z"
@@ -1225,16 +1220,11 @@ exports[`renderer/components/notifications/AccountNotifications.tsx should rende
               data-wrap="nowrap"
             >
               <span
-                class="Text-sc-17v1xeu-0 hmAFGJ"
+                class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
                 title="You created the thread."
               >
                 Authored
               </span>
-              <span
-                class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-              >
-                •
-              </span>
               <relative-time
                 class="sc-KXCwU"
                 datetime="2017-05-20T17:06:34.000Z"
@@ -1692,16 +1682,11 @@ exports[`renderer/components/notifications/AccountNotifications.tsx should rende
             data-wrap="nowrap"
           >
             <span
-              class="Text-sc-17v1xeu-0 hmAFGJ"
+              class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
               title="You're watching the repository."
             >
               Updated
             </span>
-            <span
-              class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-            >
-              •
-            </span>
             <relative-time
               class="sc-KXCwU"
               datetime="2017-05-20T17:51:57.000Z"
@@ -2036,16 +2021,11 @@ exports[`renderer/components/notifications/AccountNotifications.tsx should rende
             data-wrap="nowrap"
           >
             <span
-              class="Text-sc-17v1xeu-0 hmAFGJ"
+              class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
               title="You created the thread."
             >
               Authored
             </span>
-            <span
-              class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-            >
-              •
-            </span>
             <relative-time
               class="sc-KXCwU"
               datetime="2017-05-20T17:06:34.000Z"
diff --git a/src/renderer/components/notifications/__snapshots__/NotificationFooter.test.tsx.snap b/src/renderer/components/notifications/__snapshots__/NotificationFooter.test.tsx.snap
index 3fa98f9c9..022d35715 100644
--- a/src/renderer/components/notifications/__snapshots__/NotificationFooter.test.tsx.snap
+++ b/src/renderer/components/notifications/__snapshots__/NotificationFooter.test.tsx.snap
@@ -36,16 +36,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx security alert
           data-wrap="nowrap"
         >
           <span
-            class="Text-sc-17v1xeu-0 hmAFGJ"
+            class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
             title="You're watching the repository."
           >
             Updated
           </span>
-          <span
-            class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-          >
-            •
-          </span>
           <relative-time
             class="sc-KXCwU"
             datetime="2017-05-20T17:51:57.000Z"
@@ -166,16 +161,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx security alert
         data-wrap="nowrap"
       >
         <span
-          class="Text-sc-17v1xeu-0 hmAFGJ"
+          class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
           title="You're watching the repository."
         >
           Updated
         </span>
-        <span
-          class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-        >
-          •
-        </span>
         <relative-time
           class="sc-KXCwU"
           datetime="2017-05-20T17:51:57.000Z"
@@ -353,16 +343,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx security alert
           data-wrap="nowrap"
         >
           <span
-            class="Text-sc-17v1xeu-0 hmAFGJ"
+            class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
             title="You're watching the repository."
           >
             Updated
           </span>
-          <span
-            class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-          >
-            •
-          </span>
           <relative-time
             class="sc-KXCwU"
             datetime="2017-05-20T17:51:57.000Z"
@@ -483,16 +468,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx security alert
         data-wrap="nowrap"
       >
         <span
-          class="Text-sc-17v1xeu-0 hmAFGJ"
+          class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
           title="You're watching the repository."
         >
           Updated
         </span>
-        <span
-          class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-        >
-          •
-        </span>
         <relative-time
           class="sc-KXCwU"
           datetime="2017-05-20T17:51:57.000Z"
@@ -674,16 +654,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx should default
           data-wrap="nowrap"
         >
           <span
-            class="Text-sc-17v1xeu-0 hmAFGJ"
+            class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
             title="You're watching the repository."
           >
             Updated
           </span>
-          <span
-            class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-          >
-            •
-          </span>
           <relative-time
             class="sc-KXCwU"
             datetime="2017-05-20T17:51:57.000Z"
@@ -808,16 +783,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx should default
         data-wrap="nowrap"
       >
         <span
-          class="Text-sc-17v1xeu-0 hmAFGJ"
+          class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
           title="You're watching the repository."
         >
           Updated
         </span>
-        <span
-          class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-        >
-          •
-        </span>
         <relative-time
           class="sc-KXCwU"
           datetime="2017-05-20T17:51:57.000Z"
@@ -995,16 +965,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx should render
           data-wrap="nowrap"
         >
           <span
-            class="Text-sc-17v1xeu-0 hmAFGJ"
+            class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
             title="You're watching the repository."
           >
             Updated
           </span>
-          <span
-            class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-          >
-            •
-          </span>
           <relative-time
             class="sc-KXCwU"
             datetime="2017-05-20T17:51:57.000Z"
@@ -1125,16 +1090,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx should render
         data-wrap="nowrap"
       >
         <span
-          class="Text-sc-17v1xeu-0 hmAFGJ"
+          class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
           title="You're watching the repository."
         >
           Updated
         </span>
-        <span
-          class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-        >
-          •
-        </span>
         <relative-time
           class="sc-KXCwU"
           datetime="2017-05-20T17:51:57.000Z"
@@ -1312,16 +1272,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx should render
           data-wrap="nowrap"
         >
           <span
-            class="Text-sc-17v1xeu-0 hmAFGJ"
+            class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
             title="You're watching the repository."
           >
             Updated
           </span>
-          <span
-            class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-          >
-            •
-          </span>
           <relative-time
             class="sc-KXCwU"
             datetime="2017-05-20T17:51:57.000Z"
@@ -1442,16 +1397,11 @@ exports[`renderer/components/notifications/NotificationFooter.tsx should render
         data-wrap="nowrap"
       >
         <span
-          class="Text-sc-17v1xeu-0 hmAFGJ"
+          class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
           title="You're watching the repository."
         >
           Updated
         </span>
-        <span
-          class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-        >
-          •
-        </span>
         <relative-time
           class="sc-KXCwU"
           datetime="2017-05-20T17:51:57.000Z"
diff --git a/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap b/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap
index f1e117afa..e3e594920 100644
--- a/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap
+++ b/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap
@@ -123,16 +123,11 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
               data-wrap="nowrap"
             >
               <span
-                class="Text-sc-17v1xeu-0 hmAFGJ"
+                class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
                 title="You're watching the repository."
               >
                 Updated
               </span>
-              <span
-                class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-              >
-                •
-              </span>
               <relative-time
                 class="sc-KXCwU"
                 datetime="2017-05-20T17:51:57.000Z"
@@ -466,16 +461,11 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
             data-wrap="nowrap"
           >
             <span
-              class="Text-sc-17v1xeu-0 hmAFGJ"
+              class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
               title="You're watching the repository."
             >
               Updated
             </span>
-            <span
-              class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-            >
-              •
-            </span>
             <relative-time
               class="sc-KXCwU"
               datetime="2017-05-20T17:51:57.000Z"
@@ -830,16 +820,11 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
               data-wrap="nowrap"
             >
               <span
-                class="Text-sc-17v1xeu-0 hmAFGJ"
+                class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
                 title="You're watching the repository."
               >
                 Updated
               </span>
-              <span
-                class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-              >
-                •
-              </span>
               <relative-time
                 class="sc-KXCwU"
                 datetime="2017-05-20T17:51:57.000Z"
@@ -1137,16 +1122,11 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
             data-wrap="nowrap"
           >
             <span
-              class="Text-sc-17v1xeu-0 hmAFGJ"
+              class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
               title="You're watching the repository."
             >
               Updated
             </span>
-            <span
-              class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-            >
-              •
-            </span>
             <relative-time
               class="sc-KXCwU"
               datetime="2017-05-20T17:51:57.000Z"
@@ -1501,16 +1481,11 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
               data-wrap="nowrap"
             >
               <span
-                class="Text-sc-17v1xeu-0 hmAFGJ"
+                class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
                 title="You're watching the repository."
               >
                 Updated
               </span>
-              <span
-                class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-              >
-                •
-              </span>
               <relative-time
                 class="sc-KXCwU"
                 datetime="2017-05-20T17:51:57.000Z"
@@ -1808,16 +1783,11 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
             data-wrap="nowrap"
           >
             <span
-              class="Text-sc-17v1xeu-0 hmAFGJ"
+              class="Text-sc-17v1xeu-0 hmAFGJ pr-1"
               title="You're watching the repository."
             >
               Updated
             </span>
-            <span
-              class="Text-sc-17v1xeu-0 hmAFGJ px-1"
-            >
-              •
-            </span>
             <relative-time
               class="sc-KXCwU"
               datetime="2017-05-20T17:51:57.000Z"