fix: remove panel visibility listener on deactivate (#38)

atom-minimap · Dec 8, 2020 · 3f826be · 3f826be
1 parent e05efa5
commit 3f826be
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/minimap-find-and-replace.js b/lib/minimap-find-and-replace.js
@@ -71,10 +71,10 @@ module.exports = {
 		const [fnrPanel] = atom.workspace.getBottomPanels().filter(panel => panel.element.firstChild.classList.contains('find-and-replace'))
 
 		if (fnrPanel) {
-			this.changeVisible(true)
-			fnrPanel.onDidChangeVisible((visible) => {
+			this.changeVisible(fnrPanel.isVisible())
+			this.subscriptions.add(fnrPanel.onDidChangeVisible((visible) => {
 				this.changeVisible(visible)
-			})
+			}))
 		} else {
 			if (retry < 10) {
 				setTimeout(() => this.setOnChangeVisibility(retry + 1), 100)