feat(getTarget): throw when string not in DOM (#622)

TheSharpieOne · web-flow · commit e71c4275ecd3 · 2017-10-05T21:47:24.000-04:00
Previously it when a string target was not found in the DOM via querySelector, it would just return the `null` and bomb elswhere Now an error is thrown to help the developer track down the issue. Closes #620
diff --git a/src/utils.js b/src/utils.js
@@ -120,6 +120,9 @@ export function getTarget(target) {
     if (selection === null) {
       return document.querySelector(`#${target}`);
     }
+    if (selection === null) {
+      throw new Error(`The target '${target}' could not be identified in the dom, tip: check spelling`);
+    }
     return selection;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -120,6 +120,9 @@ export function getTarget(target) {`
`120`	`120`	`if (selection === null) {`
`121`	`121`	return document.querySelector(`#${target}`);
`122`	`122`	`}`
	`123`	`+ if (selection === null) {`
	`124`	+ throw new Error(`The target '${target}' could not be identified in the dom, tip: check spelling`);
	`125`	`+ }`
`123`	`126`	`return selection;`
`124`	`127`	`}`
`125`	`128`