Merge pull request #1 from harshmaur/hide_arrow_if_target_not_present

fix(): pierpo#126 hide arrow if target is not in the view

src/SvgArrow.js

@@ -181,6 +181,14 @@ const SvgArrow = ({
   const xStart = startingPoint.x;
   const yStart = startingPoint.y;
 
+  /**
+   * If the target is not in the view, endingPoint.x and endingPoint.y are zero.
+   * In this case, do not render the SvgArrow.
+   */
+  if (endingPoint.x === 0 && endingPoint.y === 0) {
+    return null;
+  }
+
   const endingPointWithArrow = computeEndingPointAccordingToArrowHead(
     endingPoint.x,
     endingPoint.y,

src/SvgArrow.test.js

@@ -241,5 +241,12 @@ describe('SvgArrow', () => {
         },
       });
     });
+
+    it('should not render path if target is not in the view', () => {
+      wrapper.setProps({ endingPoint: new Point(0, 0) });
+      wrapper.update();
+      const path = wrapper.find('path');
+      expect(path).toMatchObject({});
+    });
   });
 });