Merge pull request #12 from swapnilsg/master

adding null checks for e.target and e.target.parentNode
rocketium · Mar 17, 2020 · c587291 · c587291
2 parents ab2b923 + cb04e36
commit c587291
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/AlignmentGuides.js b/src/AlignmentGuides.js
@@ -125,7 +125,7 @@ class AlignmentGuides extends Component {
 	selectBox(e) {
 		const boundingBox = this.getBoundingBoxElement();
 		const boundingBoxPosition = boundingBox.current.getBoundingClientRect().toJSON();
-		if (e.target.id.indexOf('box') >= 0) {
+		if (e.target && e.target.id.indexOf('box') >= 0) {
 			const boxDimensions = e.target.getBoundingClientRect().toJSON();
 			let data = {
 				x: boxDimensions.x - boundingBoxPosition.x,
@@ -177,7 +177,7 @@ class AlignmentGuides extends Component {
 				});
 			}
 			this.props.onSelect && this.props.onSelect(e, data);
-		} else if (e.target.parentNode.id.indexOf('box') >= 0) {
+		} else if (e.target && e.target.parentNode && e.target.parentNode.id.indexOf('box') >= 0) {
 			const boxDimensions = e.target.parentNode.getBoundingClientRect().toJSON();
 			let data = {
 				x: boxDimensions.x - boundingBoxPosition.x,
@@ -233,7 +233,7 @@ class AlignmentGuides extends Component {
 	}
 
 	unSelectBox(e) {
-		if (e.target && e.target.id.indexOf('box') === -1 && e.target.parentNode.id.indexOf('box') === -1) {
+		if (e.target && e.target.id.indexOf('box') === -1 && e.target.parentNode && e.target.parentNode.id.indexOf('box') === -1) {
 			if (typeof this.props.isValidUnselect === 'function' && this.props.isValidUnselect(e) === false) {
 				return;
 			}