Merge pull request #94 from yeah/master

kate2753 · kate2753 · commit e5f389241308 · 2014-06-15T20:12:50.000-07:00
Catch exceptions caused by illegal selectors with document.querySelector
diff --git a/src/js/hopscotch.js b/src/js/hopscotch.js
@@ -359,7 +359,9 @@
         return result.length ? result[0] : null;
       }
       if (document.querySelector) {
-        return document.querySelector(target);
+        try {
+          return document.querySelector(target);
+        } catch (err) {}
       }
       // Regex test for id. Following the HTML 4 spec for valid id formats.
       // (http://www.w3.org/TR/html4/types.html#type-id)

Original file line number	Diff line number	Diff line change
`@@ -359,7 +359,9 @@`
`359`	`359`	`return result.length ? result[0] : null;`
`360`	`360`	`}`
`361`	`361`	`if (document.querySelector) {`
`362`		`- return document.querySelector(target);`
	`362`	`+ try {`
	`363`	`+ return document.querySelector(target);`
	`364`	`+ } catch (err) {}`
`363`	`365`	`}`
`364`	`366`	`// Regex test for id. Following the HTML 4 spec for valid id formats.`
`365`	`367`	`// (http://www.w3.org/TR/html4/types.html#type-id)`