From 92422f2e92a1b9d4e31c4c80ea8f12c821fc307a Mon Sep 17 00:00:00 2001
From: Roberto Guido <info@madbob.org>
Date: Thu, 16 Jul 2026 21:35:58 +0200
Subject: [PATCH] fix(ui): improve quick actions modal design

Signed-off-by: Roberto Guido <info@madbob.org>
---
 src/components/quickActions/Action.vue   | 23 +++++++++++++++++++++--
 src/components/quickActions/Settings.vue | 15 ++++++++++++---
 2 files changed, 33 insertions(+), 5 deletions(-)

diff --git a/src/components/quickActions/Action.vue b/src/components/quickActions/Action.vue
index 3079aeae68..3b510ebef1 100644
--- a/src/components/quickActions/Action.vue
+++ b/src/components/quickActions/Action.vue
@@ -5,7 +5,7 @@
 <template>
 	<div class="action">
 		<div class="action__info">
-			<DragIcon class="action__info__drag" :size="16" />
+			<DragIcon :class="'action__info__drag' + (!draggable ? ' undraggable' : '')" :size="16" />
 			<Icon class="action__info__icon" :action="action.name" />
 			<p v-if="!needsSelection">
 				{{ actionTitle }}
@@ -18,7 +18,11 @@
 				:model-value="selectedOption"
 				@update:modelValue="update" />
 		</div>
-		<NcButton :aria-label="t('mail', 'delete')" variant="tertiary-no-background" @click="$emit('delete')">
+		<NcButton
+			:aria-label="t('mail', 'delete')"
+			variant="tertiary-no-background"
+			class="delete-button"
+			@click="$emit('delete')">
 			<template #icon>
 				<CloseIcon :size="20" />
 			</template>
@@ -54,6 +58,11 @@ export default {
 			type: Object,
 			required: true,
 		},
+
+		draggable: {
+			type: Boolean,
+			required: true,
+		},
 	},
 
 	computed: {
@@ -134,13 +143,23 @@ export default {
 	align-items: center;
 	&__info{
 		display: flex;
+		flex-grow: 1;
 		&__icon{
 			margin-inline-end : 3px
 		}
 		&__drag{
 			margin-inline-end : 6px;
 			cursor: grab;
+
+			&.undraggable {
+				opacity: .2;
+				cursor: revert;
+			}
 		}
 	}
+
+	.delete-button {
+		margin-inline-start : 6px;
+	}
 }
 </style>
diff --git a/src/components/quickActions/Settings.vue b/src/components/quickActions/Settings.vue
index 20f0918649..29a6283365 100644
--- a/src/components/quickActions/Settings.vue
+++ b/src/components/quickActions/Settings.vue
@@ -43,6 +43,7 @@
 						<Action
 							:action="item"
 							:account="account"
+							:draggable="item.name !== 'deleteThread' && item.name !== 'moveThread'"
 							@update="(payload) => updateAction(payload, item)"
 							@delete="deleteAction(item)" />
 					</Draggable>
@@ -370,9 +371,10 @@ export default {
 <style lang="scss" scoped>
 
 .modal-content{
-	padding: 30px;
+	padding: 0 30px 30px 30px;
+
 	&__action{
-		padding: 9px;
+		padding: 6px 0;
 	}
 	&__save {
 		position: absolute;
@@ -382,9 +384,16 @@ export default {
 }
 
 :deep(.v-select){
-	display: flex;
+	display: grid;
+	grid-template-columns: 1fr 1fr;
+	flex-grow: 1;
     align-items: center;
     justify-content: space-between;
+	margin: 0;
+
+	.select__label {
+		margin: 0;
+	}
 }
 
 .modal-name{