From 6ef87d560e4a0cf16e77500faabb27a333b8e4e3 Mon Sep 17 00:00:00 2001
From: Steve Ruiz <steveruizok@gmail.com>
Date: Thu, 4 Mar 2021 18:28:25 +0000
Subject: [PATCH] Fixes size bug, one-point-marks bug.

---
 renderer/components/controls.tsx |  4 ++--
 renderer/lib/state.ts            | 12 +++++++++---
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/renderer/components/controls.tsx b/renderer/components/controls.tsx
index 98628c7..fcfdcc1 100644
--- a/renderer/components/controls.tsx
+++ b/renderer/components/controls.tsx
@@ -233,8 +233,8 @@ const SizeButton = styled.button<{
     display: block;
     border-radius: 100%;
     background-color: rgb(${({ color }) => color});
-    height: ${({ size }) => size / 2}px;
-    width: ${({ size }) => size / 2}px;
+    height: ${({ size }) => size * 0.75}px;
+    width: ${({ size }) => size * 0.75}px;
     transition: transform 0.12s;
     border: ${({ color }) =>
       color === "26, 28, 44"
diff --git a/renderer/lib/state.ts b/renderer/lib/state.ts
index 8bfd277..6f2b6bc 100644
--- a/renderer/lib/state.ts
+++ b/renderer/lib/state.ts
@@ -788,18 +788,23 @@ function getFreehandPath(mark: Mark, isPressure: boolean) {
 
 function getRectPath(mark: Mark) {
   const { points } = mark
+  const path = new Path2D()
+  if (points.length < 2) return path
+
   const x0 = Math.min(points[0][0], points[1][0])
   const y0 = Math.min(points[0][1], points[1][1])
   const x1 = Math.max(points[0][0], points[1][0])
   const y1 = Math.max(points[0][1], points[1][1])
 
-  const path = new Path2D()
   path.rect(x0, y0, x1 - x0, y1 - y0)
   return path
 }
 
 function getEllipsePath(mark: Mark) {
   const { points } = mark
+  const path = new Path2D()
+  if (points.length < 2) return path
+
   const x0 = Math.min(points[0][0], points[1][0])
   const y0 = Math.min(points[0][1], points[1][1])
   const x1 = Math.max(points[0][0], points[1][0])
@@ -809,13 +814,15 @@ function getEllipsePath(mark: Mark) {
   const cx = x0 + w / 2
   const cy = y0 + h / 2
 
-  const path = new Path2D()
   path.ellipse(cx, cy, w / 2, h / 2, 0, 0, Math.PI * 2)
   return path
 }
 
 function getArrowPath(mark: Mark) {
   const { points } = mark
+  const path = new Path2D()
+  if (points.length < 2) return path
+
   const [[x0, y0], [x1, y1]] = points
   const angle = Math.atan2(y1 - y0, x1 - x0)
   const distance = Math.hypot(y1 - y0, x1 - x0)
@@ -823,7 +830,6 @@ function getArrowPath(mark: Mark) {
   const [x2, y2] = projectPoint(x1, y1, angle + Math.PI * 1.2, leg)
   const [x3, y3] = projectPoint(x1, y1, angle - Math.PI * 1.2, leg)
 
-  const path = new Path2D()
   path.moveTo(x0, y0)
   path.lineTo(x1, y1)
   path.lineTo(x2, y2)