From 33bbb5d0aa247050ae1fce230969d9aba51a6cb7 Mon Sep 17 00:00:00 2001
From: Lachlan Kermode <lachiekermode@gmail.com>
Date: Thu, 4 Jun 2020 13:54:10 +0200
Subject: [PATCH] don't show non-categorised events on timeline

---
 .../presentational/Timeline/DatetimeDot.js    | 23 +++++++++++--------
 .../presentational/Timeline/Events.js         |  2 ++
 2 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/src/components/presentational/Timeline/DatetimeDot.js b/src/components/presentational/Timeline/DatetimeDot.js
index 1eff3e5..71c77c0 100644
--- a/src/components/presentational/Timeline/DatetimeDot.js
+++ b/src/components/presentational/Timeline/DatetimeDot.js
@@ -9,13 +9,16 @@ export default ({
   onSelect,
   styleProps,
   extraRender
-}) => (
-  <circle
-    onClick={onSelect}
-    className='event'
-    cx={x}
-    cy={y}
-    style={styleProps}
-    r={r}
-  />
-)
+}) => {
+  if (!y) return null
+  return (
+    <circle
+      onClick={onSelect}
+      className='event'
+      cx={x}
+      cy={y}
+      style={styleProps}
+      r={r}
+    />
+  )
+}
diff --git a/src/components/presentational/Timeline/Events.js b/src/components/presentational/Timeline/Events.js
index b9008c9..1159fec 100644
--- a/src/components/presentational/Timeline/Events.js
+++ b/src/components/presentational/Timeline/Events.js
@@ -91,6 +91,8 @@ const TimelineEvents = ({
         renderShape = renderDiamond
       } else if (event.shape === 'star') {
         renderShape = renderStar
+      } else {
+        renderShape = renderDot
       }
     }