From 42cd338db14b7e5faf824fc13c7a8e06883b3368 Mon Sep 17 00:00:00 2001 From: Artem Zhirov Date: Wed, 4 Sep 2024 01:55:39 +0500 Subject: [PATCH] Fix: prevent island re-rendering when using transition:persist (#11854) https://github.com/withastro/astro/issues/11854 --- .changeset/seven-bees-love.md | 5 +++++ packages/astro/src/transitions/swap-functions.ts | 10 +++++++++- 2 files changed, 14 insertions(+), 1 deletion(-) create mode 100644 .changeset/seven-bees-love.md diff --git a/.changeset/seven-bees-love.md b/.changeset/seven-bees-love.md new file mode 100644 index 000000000000..f1305c9822f6 --- /dev/null +++ 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 index e2d8557f5d50..c99de8d3df91 100644 --- a/packages/astro/src/transitions/swap-functions.ts +++ b/packages/astro/src/transitions/swap-functions.ts @@ -77,7 +77,11 @@ 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 +137,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,