diff --git a/packages/astro/components/ViewTransitions.astro b/packages/astro/components/ViewTransitions.astro index 822be5ccbf..4d615188db 100644 --- a/packages/astro/components/ViewTransitions.astro +++ b/packages/astro/components/ViewTransitions.astro @@ -114,7 +114,7 @@ const { fallback = 'animate' } = Astro.props as Props; const parser = new DOMParser(); // A noop element used to prevent styles from being removed - if(import.meta.env.DEV) { + if (import.meta.env.DEV) { var noopEl: string | undefined = document.createElement('div'); } @@ -145,13 +145,15 @@ const { fallback = 'animate' } = Astro.props as Props; } } // Only run this in dev. This will get stripped from production builds and is not needed. - if(import.meta.env.DEV) { - if(el.tagName === 'STYLE' && el.dataset.viteDevId) { + if (import.meta.env.DEV) { + if (el.tagName === 'STYLE' && el.dataset.viteDevId) { const devId = el.dataset.viteDevId; // If this same style tag exists, remove it from the new page - return doc.querySelector(`style[data-astro-dev-id="${devId}"]`) - // Otherwise, keep it anyways. This is client:only styles. - || noopEl; + return ( + doc.querySelector(`style[data-astro-dev-id="${devId}"]`) || + // Otherwise, keep it anyways. This is client:only styles. + noopEl + ); } } return null; diff --git a/packages/astro/e2e/view-transitions.test.js b/packages/astro/e2e/view-transitions.test.js index bd8ad5c362..64041ab05e 100644 --- a/packages/astro/e2e/view-transitions.test.js +++ b/packages/astro/e2e/view-transitions.test.js @@ -566,7 +566,7 @@ test.describe('View Transitions', () => { await expect(p, 'should have content').toHaveText('Page 1'); }); - test("client:only styles are retained on transition", async ({ page, astro }) => { + test('client:only styles are retained on transition', async ({ page, astro }) => { const totalExpectedStyles = 8; // Go to page 1