Check fuzzy-finder has contents before patching

Resolves #559.
file-icons · Apr 22, 2017 · 13dd350 · 13dd350
1 parent 9ad43da
commit 13dd350
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,9 +13,11 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - **Support:** 3D Assets (`.cginc`, `.shader`), Circle CI (`.circleci` directories), Jest (`.js.snap`), Markdown (`.mdwn`), Python (`.pyi`)
 
 ### Fixed
+- [[`#559`][]]: Occasional error thrown when opening `fuzzy-finder`
 - [[`#586`][]]: Classes not removed when disposing of `addIconToElement` disposable
 - [[`#590`][]]: Unhandled exception thrown from [`project-plus`][`Tabs`] fix.
 
+[`#559`]: https://github.com/file-icons/atom/issues/559
 [`#586`]: https://github.com/file-icons/atom/issues/586
 [`#590`]: https://github.com/file-icons/atom/issues/590
 

diff --git a/lib/consumers/fuzzy-finder.js b/lib/consumers/fuzzy-finder.js
@@ -161,6 +161,7 @@ class FuzzyFinder extends Consumer{
 			const element = this.jQueryRemoved
 				? list.element.querySelector("ol.list-group")
 				: list.list[0];
+			if(!element) return;
 			const items = Array.from(element.children);
 			for(const item of items){
 				const iconElement = item.querySelector(".primary-line.file");