From bafacf7d62630ac303211c79b252a1bc4d9d949f Mon Sep 17 00:00:00 2001
From: Grigory Vodyanov <scratchx@gmx.com>
Date: Tue, 11 Jun 2024 19:54:04 +0200
Subject: [PATCH] fix: Apply recipient search input on blur

Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
---
 src/components/Composer.vue | 59 ++++++++++++++++++++++++++-----------
 1 file changed, 41 insertions(+), 18 deletions(-)

diff --git a/src/components/Composer.vue b/src/components/Composer.vue
index cf573a3f59..7157927e9a 100644
--- a/src/components/Composer.vue
+++ b/src/components/Composer.vue
@@ -40,7 +40,7 @@
 				<NcSelect id="to"
 					ref="toLabel"
 					:value="selectTo"
-					:options="selectableRecipients.filter(reciptient=>!selectTo.some(to=>to.email===reciptient.email))"
+					:options="selectableRecipients.filter(recipient=>!selectTo.some(to=>to.email===recipient.email))"
 					:taggable="true"
 					:aria-label-combobox="t('mail', 'Select recipient')"
 					:filter-by="(option, label, search)=>filterOption(option, label, search,'to')"
@@ -52,8 +52,10 @@
 					:clearable="true"
 					:no-wrap="false"
 					:create-option="createRecipientOption"
+					:clear-search-on-blur="() => clearOnBlur('to')"
 					@input="saveDraftDebounced"
 					@option:selecting="onNewToAddr"
+					@search:blur="onNewToAddr"
 					@search="onAutocomplete($event, 'to')">
 					<template #search="{ events, attributes }">
 						<input :placeholder="t('mail', 'Contact or email address …')"
@@ -94,12 +96,12 @@
 					ref="toLabel"
 					:value="selectCc"
 					:class="{'opened': !autoLimit,'select':true}"
-					:options="selectableRecipients.filter(reciptient=>!selectCc.some(cc=>cc.email===reciptient.email))"
+					:options="selectableRecipients.filter(recipient=>!selectCc.some(cc=>cc.email===recipient.email))"
 					:no-wrap="false"
 					:filter-by="(option, label, search)=>filterOption(option, label, search,'cc')"
 					:taggable="true"
 					:close-on-select="true"
-
+					:clear-search-on-blur="() => clearOnBlur('cc')"
 					:multiple="true"
 					:placeholder="t('mail', 'Contact or email address …')"
 					:aria-label-combobox="t('mail', 'Contact or email address …')"
@@ -110,6 +112,7 @@
 					:create-option="createRecipientOption"
 					@input="saveDraftDebounced"
 					@option:selecting="onNewCcAddr"
+					@search:blur="onNewCcAddr"
 					@search="onAutocomplete($event, 'cc')">
 					<template #search="{ events, attributes }">
 						<input :placeholder="t('mail', 'Contact or email address …')"
@@ -146,10 +149,10 @@
 					:class="{'opened': !autoLimit,'select':true}"
 					:no-wrap="false"
 					:filter-by="(option, label, search)=>filterOption(option, label, search,'bcc')"
-					:options="selectableRecipients.filter(reciptient=>!selectBcc.some(bcc=>bcc.email===reciptient.email))"
+					:options="selectableRecipients.filter(recipient=>!selectBcc.some(bcc=>bcc.email===recipient.email))"
 					:taggable="true"
 					:close-on-select="true"
-
+					:clear-search-on-blur="() => clearOnBlur('bcc')"
 					:multiple="true"
 					:placeholder="t('mail', 'Contact or email address …')"
 					:aria-label-combobox="t('mail', 'Contact or email address …')"
@@ -160,6 +163,7 @@
 					:create-option="createRecipientOption"
 					@input="saveDraftDebounced"
 					@option:selecting="onNewBccAddr"
+					@search:blur="onNewBccAddr"
 					@search="onAutocomplete($event, 'bcc')">
 					<template #search="{ events, attributes }">
 						<input :placeholder="t('mail', 'Contact or email address …')"
@@ -668,6 +672,7 @@ export default {
 			wantsSmimeSign: this.smimeSign,
 			wantsSmimeEncrypt: this.smimeEncrypt,
 			isPickerOpen: false,
+			recipientSearchTerms: {},
 		}
 	},
 	computed: {
@@ -969,6 +974,9 @@ export default {
 		window.removeEventListener('mailvelope', this.onMailvelopeLoaded)
 	},
 	methods: {
+		clearOnBlur(event) {
+			return this.recipientSearchTerms[event].includes('@')
+		},
 		handleShow(event) {
 			this.$emit('show-toolbar', event)
 		},
@@ -1174,19 +1182,20 @@ export default {
 		onAddCloudAttachmentLink() {
 			this.bus.emit('on-add-cloud-attachment-link')
 		},
-		onAutocomplete(term, loadingIndicator) {
+		onAutocomplete(term, addressType) {
 			if (term === undefined || term === '') {
 				return
 			}
-			this.loadingIndicatorTo = loadingIndicator === 'to'
-			this.loadingIndicatorCc = loadingIndicator === 'cc'
-			this.loadingIndicatorBcc = loadingIndicator === 'bcc'
+			this.loadingIndicatorTo = addressType === 'to'
+			this.loadingIndicatorCc = addressType === 'cc'
+			this.loadingIndicatorBcc = addressType === 'bcc'
+			this.recipientSearchTerms[addressType] = term
 			debouncedSearch(term).then((results) => {
-				if (loadingIndicator === 'to') {
+				if (addressType === 'to') {
 					this.loadingIndicatorTo = false
-				} else if (loadingIndicator === 'cc') {
+				} else if (addressType === 'cc') {
 					this.loadingIndicatorCc = false
-				} else if (loadingIndicator === 'bcc') {
+				} else if (addressType === 'bcc') {
 					this.loadingIndicatorBcc = false
 				}
 
@@ -1212,16 +1221,30 @@ export default {
 			await this.checkRecipientsKeys()
 		},
 		onNewToAddr(option) {
-			this.onNewAddr(option, this.selectTo)
+			this.onNewAddr(option, this.selectTo, 'to')
 		},
 		onNewCcAddr(option) {
-			this.onNewAddr(option, this.selectCc)
+			this.onNewAddr(option, this.selectCc, 'cc')
 		},
 		onNewBccAddr(option) {
-			this.onNewAddr(option, this.selectBcc)
-		},
-		onNewAddr(option, list) {
-			if (list.some((recipient) => recipient.email === option.email)) {
+			this.onNewAddr(option, this.selectBcc, 'bcc')
+		},
+		onNewAddr(option, list, type) {
+			if (
+				(option === null || option === undefined)
+				&& this.recipientSearchTerms[type] !== undefined
+				&& this.recipientSearchTerms[type] !== ''
+			) {
+				if (!this.recipientSearchTerms[type].includes('@')) {
+					return
+				}
+				option = {}
+				option.email = this.recipientSearchTerms[type]
+				option.label = this.recipientSearchTerms[type]
+				this.recipientSearchTerms[type] = ''
+			}
+
+			if (list.some((recipient) => recipient.email === option?.email) || !option) {
 				return
 			}
 			const recipient = { ...option }