From f1ef4f9f37f465d8bc223adc731bf75b1a741502 Mon Sep 17 00:00:00 2001
From: kolaente <k@knt.li>
Date: Thu, 28 Jan 2021 23:02:11 +0100
Subject: [PATCH] Fix opening multiple popups

---
 .../list/list-settings-dropdown.vue           | 22 +++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/src/components/list/list-settings-dropdown.vue b/src/components/list/list-settings-dropdown.vue
index 6e51dbdb3..1feecf427 100644
--- a/src/components/list/list-settings-dropdown.vue
+++ b/src/components/list/list-settings-dropdown.vue
@@ -1,7 +1,7 @@
 <template>
-	<div class="dropdown is-right is-active">
+	<div class="dropdown is-right is-active" ref="dropdown">
 		<div class="dropdown-trigger">
-			<icon icon="ellipsis-h" @click.stop="listSettingsOpen = !listSettingsOpen" class="icon"/>
+			<icon icon="ellipsis-h" @click="listSettingsOpen = !listSettingsOpen" class="icon"/>
 		</div>
 		<transition name="fade">
 			<div class="dropdown-menu" v-if="listSettingsOpen">
@@ -65,6 +65,7 @@
 </template>
 
 <script>
+import {closeWhenClickedOutside} from '@/helpers/closeWhenClickedOutside'
 import {getSavedFilterIdFromListId} from '@/helpers/savedFilter'
 
 export default {
@@ -74,8 +75,11 @@ export default {
 			listSettingsOpen: false,
 		}
 	},
-	created() {
-		this.$nextTick(() => document.addEventListener('click', () => this.listSettingsOpen = false))
+	mounted() {
+		document.addEventListener('click', this.hide)
+	},
+	beforeDestroy() {
+		document.removeEventListener('click', this.hide)
 	},
 	props: {
 		list: {
@@ -103,5 +107,15 @@ export default {
 			return getSavedFilterIdFromListId(this.list.id) > 0
 		},
 	},
+	methods: {
+		hide(e) {
+			if (this.listSettingsOpen) {
+				console.log('hide', e)
+				closeWhenClickedOutside(e, this.$refs.dropdown, () => {
+					this.listSettingsOpen = false
+				})
+			}
+		},
+	},
 }
 </script>