From 3f02e854cb52891178f3c8f58479cd91e4ec2b84 Mon Sep 17 00:00:00 2001
From: greta <gretadoci@gmail.com>
Date: Fri, 12 Sep 2025 17:41:21 +0200
Subject: [PATCH 1/5] fix: thread scrolling position

Signed-off-by: greta <gretadoci@gmail.com>
---
 src/components/ThreadEnvelope.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/ThreadEnvelope.vue b/src/components/ThreadEnvelope.vue
index 37052e698c..1d4cd8b9e9 100644
--- a/src/components/ThreadEnvelope.vue
+++ b/src/components/ThreadEnvelope.vue
@@ -737,7 +737,7 @@ export default {
 			this.$nextTick(() => {
 				const threadElement = document.querySelector(`[data-thread-id="${threadId}"]`)
 				if (threadElement) {
-					threadElement.scrollIntoView({ behavior: 'smooth', block: 'top' })
+					threadElement.scrollIntoView({ behavior: 'smooth', block: 'start' })
 				}
 			})
 		},
@@ -746,7 +746,7 @@ export default {
 			this.$nextTick(() => {
 				const envelopeElement = this.$refs.envelope
 				if (envelopeElement) {
-					envelopeElement.scrollIntoView({ behavior: 'smooth', block: 'top' })
+					envelopeElement.scrollIntoView({ behavior: 'smooth', block: 'start' })
 				}
 			})
 		},

From 723aa6c29b27f7f583bd8f751f605f3c4d48c29c Mon Sep 17 00:00:00 2001
From: greta <gretadoci@gmail.com>
Date: Mon, 15 Sep 2025 13:07:09 +0200
Subject: [PATCH 2/5] fixup! fix: thread scrolling position

Signed-off-by: greta <gretadoci@gmail.com>
---
 src/components/ThreadEnvelope.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/ThreadEnvelope.vue b/src/components/ThreadEnvelope.vue
index 1d4cd8b9e9..16fd0e11c5 100644
--- a/src/components/ThreadEnvelope.vue
+++ b/src/components/ThreadEnvelope.vue
@@ -737,7 +737,7 @@ export default {
 			this.$nextTick(() => {
 				const threadElement = document.querySelector(`[data-thread-id="${threadId}"]`)
 				if (threadElement) {
-					threadElement.scrollIntoView({ behavior: 'smooth', block: 'start' })
+					threadElement.scrollIntoView({ behavior: 'smooth', block: '' })
 				}
 			})
 		},
@@ -746,7 +746,7 @@ export default {
 			this.$nextTick(() => {
 				const envelopeElement = this.$refs.envelope
 				if (envelopeElement) {
-					envelopeElement.scrollIntoView({ behavior: 'smooth', block: 'start' })
+					envelopeElement.scrollIntoView({ behavior: 'smooth', block: '' })
 				}
 			})
 		},

From fc9a9a28d55164f55f0030ce27ba3a32c52656f1 Mon Sep 17 00:00:00 2001
From: greta <gretadoci@gmail.com>
Date: Mon, 15 Sep 2025 13:33:50 +0200
Subject: [PATCH 3/5] fixup! fix: thread scrolling position

Signed-off-by: greta <gretadoci@gmail.com>
---
 src/components/ThreadEnvelope.vue | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/components/ThreadEnvelope.vue b/src/components/ThreadEnvelope.vue
index 16fd0e11c5..246fcb4738 100644
--- a/src/components/ThreadEnvelope.vue
+++ b/src/components/ThreadEnvelope.vue
@@ -735,19 +735,13 @@ export default {
 		},
 		scrollToThread(threadId) {
 			this.$nextTick(() => {
-				const threadElement = document.querySelector(`[data-thread-id="${threadId}"]`)
-				if (threadElement) {
-					threadElement.scrollIntoView({ behavior: 'smooth', block: '' })
-				}
+				window.scrollTo({ top: 0, behavior: 'smooth' })
 			})
 		},
 
 		scrollToEnvelope() {
 			this.$nextTick(() => {
-				const envelopeElement = this.$refs.envelope
-				if (envelopeElement) {
-					envelopeElement.scrollIntoView({ behavior: 'smooth', block: '' })
-				}
+				window.scrollTo({ top: 0, behavior: 'smooth' })
 			})
 		},
 		async fetchItineraries() {

From c77d9b6457259aa6aced95a520982e4f50fc2238 Mon Sep 17 00:00:00 2001
From: greta <gretadoci@gmail.com>
Date: Thu, 2 Oct 2025 12:42:54 +0200
Subject: [PATCH 4/5] fixup! fix: thread scrolling position

Signed-off-by: greta <gretadoci@gmail.com>
---
 src/components/ThreadEnvelope.vue | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/components/ThreadEnvelope.vue b/src/components/ThreadEnvelope.vue
index 246fcb4738..76f7fd6b82 100644
--- a/src/components/ThreadEnvelope.vue
+++ b/src/components/ThreadEnvelope.vue
@@ -734,15 +734,17 @@ export default {
 			}
 		},
 		scrollToThread(threadId) {
-			this.$nextTick(() => {
-				window.scrollTo({ top: 0, behavior: 'smooth' })
-			})
+			const threadElement = document.querySelector(`[data-thread-id="${threadId}"]`)
+			if (threadElement) {
+				threadElement.scrollIntoView({ behavior: 'smooth', block: 'start' })
+			}
 		},
 
 		scrollToEnvelope() {
-			this.$nextTick(() => {
-				window.scrollTo({ top: 0, behavior: 'smooth' })
-			})
+			const envelopeElement = this.$refs.envelope
+			if (envelopeElement) {
+				envelopeElement.scrollIntoView({ behavior: 'smooth', block: 'start' })
+			}
 		},
 		async fetchItineraries() {
 			// Sanity check before actually making the request

From 86c7de0221d9b78ed70117dfe56a6db6ffe951f9 Mon Sep 17 00:00:00 2001
From: greta <gretadoci@gmail.com>
Date: Wed, 8 Oct 2025 17:45:50 +0200
Subject: [PATCH 5/5] fixup! fix: thread scrolling position

Signed-off-by: greta <gretadoci@gmail.com>
---
 src/components/ThreadEnvelope.vue | 89 ++++++++++++++++---------------
 1 file changed, 47 insertions(+), 42 deletions(-)

diff --git a/src/components/ThreadEnvelope.vue b/src/components/ThreadEnvelope.vue
index 76f7fd6b82..721c879489 100644
--- a/src/components/ThreadEnvelope.vue
+++ b/src/components/ThreadEnvelope.vue
@@ -18,7 +18,6 @@
 				</NcButton>
 			</div>
 		</div>
-
 		<div class="envelope__header">
 			<div class="envelope__header__avatar">
 				<Avatar v-if="envelope.from && envelope.from[0]"
@@ -46,7 +45,6 @@
 					:data-starred="envelope.flags.$junk ? 'true' : 'false'"
 					@click.prevent="hasWriteAcl ? onToggleJunk() : false" />
 			</div>
-
 			<router-link :to="route"
 				event=""
 				class="left"
@@ -228,46 +226,48 @@
 				</template>
 			</div>
 		</div>
-		<MessageLoadingSkeleton v-if="loading === Loading.Skeleton" />
-		<Message v-if="message"
-			v-show="loading === Loading.Done"
-			:envelope="envelope"
-			:message="message"
-			:full-height="fullHeight"
-			:smart-replies="showFollowUpHeader ? [] : smartReplies"
-			:reply-button-label="replyButtonLabel"
-			@load="onMessageLoaded"
-			@translate="onOpenTranslationModal"
-			@reply="(body) => onReply(body, showFollowUpHeader)" />
-		<Error v-else-if="error"
-			:error="error.message || t('mail', 'Not found')"
-			message=""
-			:data="error"
-			:auto-margin="true"
-			role="alert" />
-		<ConfirmModal v-if="message && message.unsubscribeUrl && message.isOneClickUnsubscribe && showListUnsubscribeConfirmation"
-			:confirm-text="t('mail', 'Unsubscribe')"
-			:title="t('mail', 'Unsubscribe via link')"
-			@cancel="showListUnsubscribeConfirmation = false"
-			@confirm="unsubscribeViaOneClick">
-			{{ t('mail', 'Unsubscribing will stop all messages from the mailing list {sender}', { sender: from }) }}
-		</ConfirmModal>
-		<ConfirmModal v-else-if="message && message.unsubscribeUrl && showListUnsubscribeConfirmation"
-			:confirm-text="t('mail', 'Unsubscribe')"
-			:confirm-url="message.unsubscribeUrl"
-			:title="t('mail', 'Unsubscribe via link')"
-			@cancel="showListUnsubscribeConfirmation = false"
-			@confirm="showListUnsubscribeConfirmation = false">
-			{{ t('mail', 'Unsubscribing will stop all messages from the mailing list {sender}', { sender: from }) }}
-		</ConfirmModal>
-		<ConfirmModal v-else-if="message && message.unsubscribeMailto && showListUnsubscribeConfirmation"
-			:confirm-text="t('mail', 'Send unsubscribe email')"
-			:title="t('mail', 'Unsubscribe via email')"
-			:disabled="unsubscribing"
-			@cancel="showListUnsubscribeConfirmation = false"
-			@confirm="unsubscribeViaMailto">
-			{{ t('mail', 'Unsubscribing will stop all messages from the mailing list {sender}', { sender: from }) }}
-		</ConfirmModal>
+		<div class="envelope__body-container">
+			<MessageLoadingSkeleton v-if="loading === Loading.Skeleton" />
+			<Message v-if="message"
+				v-show="loading === Loading.Done"
+				:envelope="envelope"
+				:message="message"
+				:full-height="fullHeight"
+				:smart-replies="showFollowUpHeader ? [] : smartReplies"
+				:reply-button-label="replyButtonLabel"
+				@load="onMessageLoaded"
+				@translate="onOpenTranslationModal"
+				@reply="(body) => onReply(body, showFollowUpHeader)" />
+			<Error v-else-if="error"
+				:error="error.message || t('mail', 'Not found')"
+				message=""
+				:data="error"
+				:auto-margin="true"
+				role="alert" />
+			<ConfirmModal v-if="message && message.unsubscribeUrl && message.isOneClickUnsubscribe && showListUnsubscribeConfirmation"
+				:confirm-text="t('mail', 'Unsubscribe')"
+				:title="t('mail', 'Unsubscribe via link')"
+				@cancel="showListUnsubscribeConfirmation = false"
+				@confirm="unsubscribeViaOneClick">
+				{{ t('mail', 'Unsubscribing will stop all messages from the mailing list {sender}', { sender: from }) }}
+			</ConfirmModal>
+			<ConfirmModal v-else-if="message && message.unsubscribeUrl && showListUnsubscribeConfirmation"
+				:confirm-text="t('mail', 'Unsubscribe')"
+				:confirm-url="message.unsubscribeUrl"
+				:title="t('mail', 'Unsubscribe via link')"
+				@cancel="showListUnsubscribeConfirmation = false"
+				@confirm="showListUnsubscribeConfirmation = false">
+				{{ t('mail', 'Unsubscribing will stop all messages from the mailing list {sender}', { sender: from }) }}
+			</ConfirmModal>
+			<ConfirmModal v-else-if="message && message.unsubscribeMailto && showListUnsubscribeConfirmation"
+				:confirm-text="t('mail', 'Send unsubscribe email')"
+				:title="t('mail', 'Unsubscribe via email')"
+				:disabled="unsubscribing"
+				@cancel="showListUnsubscribeConfirmation = false"
+				@confirm="unsubscribeViaMailto">
+				{{ t('mail', 'Unsubscribing will stop all messages from the mailing list {sender}', { sender: from }) }}
+			</ConfirmModal>
+		</div>
 	</div>
 </template>
 <script>
@@ -1041,6 +1041,11 @@ export default {
 				transform: none;
 			}
 		}
+		&__body-container {
+			flex: 1;
+			overflow-y: auto;
+			overflow-x: hidden;
+		}
 
 		& + .envelope {
 			margin-top: calc(var(--default-grid-baseline) * -7);