From a7bbf77fd6f82db0b7a04724cfab4ba81008272a Mon Sep 17 00:00:00 2001
From: Luka Trovic <luka@nextcloud.com>
Date: Fri, 9 Jun 2023 17:55:39 +0200
Subject: [PATCH] feat: update layout of translate dialog

Signed-off-by: Luka Trovic <luka@nextcloud.com>
---
 src/components/Menu/MenuBar.vue    |   8 ++-
 src/components/Modal/Translate.vue | 103 ++++++++++++++++++++++++-----
 2 files changed, 93 insertions(+), 18 deletions(-)

diff --git a/src/components/Menu/MenuBar.vue b/src/components/Menu/MenuBar.vue
index c2052735839..a510a935747 100644
--- a/src/components/Menu/MenuBar.vue
+++ b/src/components/Menu/MenuBar.vue
@@ -247,7 +247,13 @@ export default {
 		},
 		showTranslate() {
 			const { from, to } = this.$editor.view.state.selection
-			const selectedText = this.$editor.view.state.doc.textBetween(from, to, ' ')
+			let selectedText = this.$editor.view.state.doc.textBetween(from, to, ' ')
+
+			if (!selectedText.trim().length) {
+				this.$editor.commands.selectAll()
+				selectedText = this.$editor.view.state.doc.textContent
+			}
+
 			console.debug('translation click', this.$editor.view.state.selection, selectedText)
 			this.displayTranslate = selectedText ?? ''
 		},
diff --git a/src/components/Modal/Translate.vue b/src/components/Modal/Translate.vue
index 47c11c9be08..197805148bc 100644
--- a/src/components/Modal/Translate.vue
+++ b/src/components/Modal/Translate.vue
@@ -20,26 +20,43 @@
   -->
 
 <template>
-	<NcModal @close="$emit('close')">
+	<NcModal size="large" @close="$emit('close')">
 		<div class="translate-dialog">
 			<h2>{{ t('text', 'Translate') }}</h2>
-			<textarea v-model="input" />
-			<div class="language-selector">
-				<label for="fromLanguage">{{ t('text', 'Source language') }}</label>
-				<NcSelect v-model="fromLanguage"
-					input-id="fromLanguage"
-					:options="fromLanguages"
-					:append-to-body="false" />
+			<em>{{ t('text', 'To translate individual parts of the text, select it before using the translate function.') }}</em>
+			<div class="wrapper">
+				<div class="col">
+					<div class="language-selector">
+						<label for="fromLanguage">{{ t('text', 'Translate from') }}</label>
+						<NcSelect v-model="fromLanguage"
+							input-id="fromLanguage"
+							:placeholder="t('text', 'Select language')"
+							:options="fromLanguages"
+							:disabled="disableFromLanguageSelect"
+							:append-to-body="false" />
+					</div>
+					<textarea ref="input"
+						v-model="input"
+						autofocus
+						@input="autosize"
+						@focus="onInputFocus" />
+				</div>
+				<div class="col">
+					<div class="language-selector">
+						<label for="toLanguage">{{ t('text', 'to') }}</label>
+						<NcSelect v-model="toLanguage"
+							input-id="toLanguage"
+							:placeholder="t('text', 'Select language')"
+							:options="toLanguages"
+							:disabled="!fromLanguage"
+							:append-to-body="false" />
+					</div>
+					<textarea ref="result"
+						v-model="result"
+						readonly
+						:class="{'icon-loading': loading }" />
+				</div>
 			</div>
-			<div class="language-selector">
-				<label for="toLanguage">{{ t('text', 'Target language') }}</label>
-				<NcSelect v-model="toLanguage"
-					input-id="toLanguage"
-					:options="toLanguages"
-					:disabled="!fromLanguage"
-					:append-to-body="false" />
-			</div>
-			<textarea v-model="result" readonly :class="{'icon-loading': loading }" />
 			<div class="translate-actions">
 				<NcLoadingIcon v-if="loading" />
 				<NcButton v-if="!result"
@@ -94,6 +111,7 @@ export default {
 			canDetect: loadState('text', 'translation_can_detect'),
 			loading: false,
 			error: null,
+			disableFromLanguageSelect: true,
 		}
 	},
 	computed: {
@@ -172,6 +190,18 @@ export default {
 		async contentReplace() {
 			this.$emit('replace-content', this.result)
 		},
+		autosize() {
+			this.$refs.input.style.overflowY = 'hidden'
+			this.$refs.input.style.height = 'auto'
+			const height = this.$refs.input.scrollHeight + 10
+			this.$refs.input.style.height = height + 'px'
+			this.$refs.result.style.height = height + 'px'
+			this.$refs.input.style.overflowY = 'auto'
+		},
+		onInputFocus() {
+			this.disableFromLanguageSelect = false
+			this.autosize()
+		},
 	},
 }
 </script>
@@ -179,13 +209,52 @@ export default {
 <style lang="scss" scoped>
 .translate-dialog {
 	margin: 24px;
+
+	.wrapper {
+		display: grid;
+		grid-template-columns: repeat(auto-fill, minmax(300px, 1fr));
+		grid-column-gap: 16px;
+		margin-top: calc(var(--default-grid-baseline) * 6);
+
+		.language-selector {
+			font-weight: bold;
+			flex-wrap: wrap;
+			gap: var(--default-grid-baseline);
+		}
+
+		.col {
+			grid-row: 1/2;
+		}
+	}
 }
 
 textarea {
 	display: block;
 	width: 100%;
 	margin-bottom: 12px;
+	height: auto;
+	resize: none;
+	box-sizing: border-box;
+	overflow-y: auto;
+	max-height: 60vh;
+}
+
+@media (max-width: 670px) {
+	.translate-dialog {
+		.wrapper {
+			display: block;
+		}
+
+		.language-selector {
+			.select {
+				width: 100%;
+			}
+		}
+	}
 
+	textarea {
+		max-height: 20vh;
+	}
 }
 
 .language-selector {