Fix: prevent island re-rendering when using transition:persist (witha…

…stro#11854) withastro#11854
azhirov · Sep 3, 2024 · 1107784 · 1107784
1 parent 5d7bc70
commit 1107784
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/.changeset/seven-bees-love.md b/.changeset/seven-bees-love.md
@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Fix: prevent island from re-rendering when using transition:persist (#11854)
diff --git a/packages/astro/src/transitions/swap-functions.ts b/packages/astro/src/transitions/swap-functions.ts
@@ -77,7 +77,7 @@ export function swapBodyElement(newElement: Element, oldElement: Element) {
 			// from the old page so that state is preserved.
 			newEl.replaceWith(el);
 			// For islands, copy over the props to allow them to re-render
-			if (newEl.localName === 'astro-island' && shouldCopyProps(el as HTMLElement)) {
+			if (newEl.localName === 'astro-island' && shouldCopyProps(el as HTMLElement) && !isSameProps(el, newEl)) {
 				el.setAttribute('ssr', '');
 				el.setAttribute('props', newEl.getAttribute('props')!);
 			}
@@ -133,6 +133,10 @@ const shouldCopyProps = (el: HTMLElement): boolean => {
 	return persistProps == null || persistProps === 'false';
 };
 
+const isSameProps = (oldEl: Element, newEl: Element) => {
+	return oldEl.getAttribute('props') === newEl.getAttribute('props')
+}
+
 export const swapFunctions = {
 	deselectScripts,
 	swapRootAttributes,