better way to add separator

kitconcept · Jan 29, 2025 · 91b08ab · 91b08ab
1 parent a23e6c0
commit 91b08ab
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/packages/volto-light-theme/src/components/Summary/EventSummary.jsx b/packages/volto-light-theme/src/components/Summary/EventSummary.jsx
@@ -13,10 +13,10 @@ const EventSummary = (props) => {
       </span>
     ),
     item.head_title,
-  ].filter((x) => x);
-  if (kicker.length > 1) {
-    kicker = kicker.reduce((prev, curr) => [prev, ' | ', curr]);
-  }
+  ]
+    .filter((x) => x)
+    .flatMap((x) => [' | ', x])
+    .slice(1);
 
   return (
     <>

diff --git a/packages/volto-light-theme/src/components/Summary/NewsItemSummary.jsx b/packages/volto-light-theme/src/components/Summary/NewsItemSummary.jsx
@@ -6,17 +6,17 @@ const NewsItemSummary = (props) => {
     day: 'numeric',
   });
 
-  let kicker = [
+  const kicker = [
     item.EffectiveDate !== 'None' && item.effective && (
       <span className="day" key="day" suppressHydrationWarning>
         {formatter.format(new Date(item.effective))}
       </span>
     ),
     item.head_title,
-  ].filter((x) => x);
-  if (kicker.length > 1) {
-    kicker = kicker.reduce((prev, curr) => [prev, ' | ', curr]);
-  }
+  ]
+    .filter((x) => x)
+    .flatMap((x) => [' | ', x])
+    .slice(1);
 
   return (
     <>