From e7d17f246914e533ffe9896790ea0e13a2f9144e Mon Sep 17 00:00:00 2001
From: Candid Dauth <cdauth@cdauth.eu>
Date: Tue, 14 Nov 2023 21:25:14 +0100
Subject: [PATCH] Fix modal styles when attributes change

---
 .../src/lib/components/ui/hybrid-popover.vue   | 18 ++++++++++++++----
 .../src/lib/components/ui/modal-dialog.vue     | 11 +++++++----
 2 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/frontend/src/lib/components/ui/hybrid-popover.vue b/frontend/src/lib/components/ui/hybrid-popover.vue
index db2bf68b..a2e934a5 100644
--- a/frontend/src/lib/components/ui/hybrid-popover.vue
+++ b/frontend/src/lib/components/ui/hybrid-popover.vue
@@ -6,11 +6,12 @@
 </script>
 
 <script setup lang="ts">
-	import { ref, watch } from "vue";
+	import { ref, toRef, watch } from "vue";
 	import { useModal } from "../../utils/modal";
 	import { useMaxBreakpoint } from "../../utils/bootstrap";
 	import Popover from "./popover.vue";
 	import { useRefWithOverride } from "../../utils/vue";
+	import AttributePreservingElement from "./attribute-preserving-element.vue";
 
 	const props = withDefaults(defineProps<{
 		show?: boolean;
@@ -54,7 +55,8 @@
 
 	const trigger = ref<HTMLElement>();
 
-	const modalRef = ref<HTMLElement>();
+	const modalElementRef = ref<InstanceType<typeof AttributePreservingElement>>();
+	const modalRef = toRef(() => modalElementRef.value?.elementRef);
 	const modal = useModal(modalRef, {
 		onShown: () => {
 			emit("shown");
@@ -109,7 +111,15 @@
 		</Popover>
 
 		<Teleport to="body">
-			<div v-if="showModal" class="modal fade" :class="props.customClass" tabindex="-1" aria-hidden="true" ref="modalRef">
+			<AttributePreservingElement
+				v-if="showModal"
+				tag="div"
+				class="modal fade"
+				:class="props.customClass"
+				tabindex="-1"
+				aria-hidden="true"
+				ref="modalElementRef"
+			>
 				<div class="modal-dialog modal-dialog-scrollable">
 					<div class="modal-content">
 						<div v-if="props.title" class="modal-header">
@@ -124,7 +134,7 @@
 						</div>
 					</div>
 				</div>
-			</div>
+			</AttributePreservingElement>
 		</Teleport>
 	</div>
 </template>
\ No newline at end of file
diff --git a/frontend/src/lib/components/ui/modal-dialog.vue b/frontend/src/lib/components/ui/modal-dialog.vue
index a29ab569..a23cb019 100644
--- a/frontend/src/lib/components/ui/modal-dialog.vue
+++ b/frontend/src/lib/components/ui/modal-dialog.vue
@@ -4,6 +4,7 @@
 	import ValidatedForm, { type CustomSubmitEvent } from "./validated-form/validated-form.vue";
 	import type { ThemeColour } from "../../utils/bootstrap";
 	import { useUnloadHandler } from "../../utils/utils";
+	import AttributePreservingElement from "./attribute-preserving-element.vue";
 
 	const props = withDefaults(defineProps<{
 		title?: string;
@@ -35,7 +36,8 @@
 	const isSubmitting = computed(() => validatedFormRef.value?.formData.isSubmitting);
 	const submitRef = ref<HTMLElement>();
 
-	const modalRef = ref<HTMLElement>();
+	const modalElementRef = ref<InstanceType<typeof AttributePreservingElement>>();
+	const modalRef = toRef(() => modalElementRef.value?.elementRef);
 	const modal = useModal(modalRef, {
 		onShown: () => {
 			const focusEl = (
@@ -78,7 +80,8 @@
 
 <template>
 	<Teleport to="body">
-		<div
+		<AttributePreservingElement
+			tag="div"
 			class="modal fade fm-modal"
 			:class="[
 				props.size !== 'default' ? `modal-${props.size}` : undefined,
@@ -86,7 +89,7 @@
 			]"
 			tabindex="-1"
 			aria-hidden="true"
-			ref="modalRef"
+			ref="modalElementRef"
 		>
 			<div class="modal-dialog modal-dialog-scrollable">
 				<ValidatedForm
@@ -133,7 +136,7 @@
 					</div>
 				</ValidatedForm>
 			</div>
-		</div>
+		</AttributePreservingElement>
 	</Teleport>
 </template>