From d99fb0d2db15c22bd38966cbfdb048c0dd5271a1 Mon Sep 17 00:00:00 2001
From: Lamont Crook <lamont@adobe.com>
Date: Wed, 8 Jul 2026 14:01:50 -0600
Subject: [PATCH 1/4] Refactor hero into a variant-driven video backdrop with
 poster

The `video` variant now uses the authored image as the LCP poster and layers
an authored .mp4/.webm URL on top as a muted, looping, autoplay backdrop. The
poster is loaded eagerly at high priority; the video is attached only after the
page load event to protect LCP, and is skipped entirely under
prefers-reduced-motion or when autoplay is blocked (poster remains). Video is
now gated on the .video variant instead of URL-guessing heuristics; the
YouTube/Vimeo embed paths were removed in favor of direct video files.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 blocks/hero/hero.css |  15 ++--
 blocks/hero/hero.js  | 178 ++++++++++++++++++++++++-------------------
 2 files changed, 104 insertions(+), 89 deletions(-)

diff --git a/blocks/hero/hero.css b/blocks/hero/hero.css
index 83f3ece..52d63ba 100644
--- a/blocks/hero/hero.css
+++ b/blocks/hero/hero.css
@@ -26,17 +26,14 @@
   object-fit: cover;
 }
 
-.hero .hero-video-embed {
-  width: 100%;
-  height: 100%;
+/* video layers over the poster image and fades in once it is playing */
+.hero .hero-video {
+  opacity: 0;
+  transition: opacity 0.6s ease;
 }
 
-.hero .hero-video-embed iframe {
-  width: 100%;
-  height: 100%;
-  object-fit: cover;
-  position: absolute;
-  inset: 0;
+.hero .hero-video.is-playing {
+  opacity: 1;
 }
 
 /* Overlay gradient */
diff --git a/blocks/hero/hero.js b/blocks/hero/hero.js
index 86db8d5..1263b55 100644
--- a/blocks/hero/hero.js
+++ b/blocks/hero/hero.js
@@ -1,107 +1,125 @@
 /**
- * Hero Block — supports both image and video backgrounds (full width).
+ * Hero Block
  *
- * Authoring structure:
- * Row 1: image OR video (as link, plain URL text, or <video> element)
- * Row 2+: heading, subtext, CTA
+ * Default: a full-width background image with overlaid heading, subtext and CTA.
  *
- * Supported video formats:
- * - A link to .mp4 file: <a href="video.mp4">...</a>
- * - Plain text URL ending in .mp4
- * - A link with youtube.com or vimeo.com
- * - An already-rendered <video> element
+ * `video` variant (authored as "Hero (video)"): the authored image becomes the
+ * LCP poster and an authored video URL (.mp4/.webm) is layered on top as a
+ * muted, looping, autoplay backdrop. To protect LCP the video is only loaded
+ * after the page has finished loading, and it is skipped entirely when the user
+ * prefers reduced motion or autoplay is blocked — the poster image remains.
  *
- * @param {Element} block The block element
+ * Authoring (video variant), a single cell:
+ *   image (poster)
+ *   video URL (.mp4 or .webm, as a link or plain text)
+ *   heading / subtext / CTA
  */
-export default async function decorate(block) {
-  const rows = [...block.children];
-  if (rows.length === 0) return;
 
-  const firstRow = rows[0];
-  let mediaEl;
-  // original node that becomes the media, so we can exclude it from the text content
-  let mediaSource;
+const VIDEO_URL_RE = /^https?:\/\/\S+\.(mp4|webm)(\?\S*)?$/i;
 
-  // Detect video/image source anywhere in the block (media may share a row with text)
-  const videoEl = block.querySelector('video');
-  const videoLink = block.querySelector('a[href$=".mp4"], a[href$=".webm"], a[href*="youtube"], a[href*="youtu.be"], a[href*="vimeo"]');
-  const picture = block.querySelector('picture');
+/**
+ * Layers a muted, looping, autoplay video over the poster image. The video is
+ * only revealed once it is actually playing, and removed if autoplay is blocked
+ * so the poster image stays visible.
+ * @param {Element} block The hero block
+ * @param {string} url The video source URL
+ */
+function loadHeroVideo(block, url) {
+  const [, ext] = url.match(/\.(mp4|webm)/i);
+  const video = document.createElement('video');
+  video.className = 'hero-media hero-video';
+  video.muted = true;
+  video.loop = true;
+  video.autoplay = true;
+  video.playsInline = true;
+  video.setAttribute('muted', '');
+  video.setAttribute('loop', '');
+  video.setAttribute('autoplay', '');
+  video.setAttribute('playsinline', '');
+  video.setAttribute('preload', 'auto');
+  video.setAttribute('aria-hidden', 'true');
+  video.tabIndex = -1;
 
-  // Check for plain text URL (no <a> tag, just pasted URL text)
-  let plainVideoUrl = null;
-  if (!videoEl && !videoLink && !picture) {
-    const text = firstRow.textContent.trim();
-    if (text.match(/https?:\/\/.*\.(mp4|webm)/i)) {
-      plainVideoUrl = text;
-    } else if (text.match(/https?:\/\/.*(youtube|youtu\.be|vimeo)/i)) {
-      plainVideoUrl = text;
-    }
+  const source = document.createElement('source');
+  source.src = url;
+  source.type = `video/${ext.toLowerCase()}`;
+  video.append(source);
+
+  // reveal only once it is genuinely playing, so we never flash a black frame
+  video.addEventListener('playing', () => video.classList.add('is-playing'), { once: true });
+
+  // insert above the poster image but below the overlay/content
+  block.insertBefore(video, block.querySelector('.hero-content'));
+
+  const playPromise = video.play();
+  if (playPromise && typeof playPromise.catch === 'function') {
+    // autoplay blocked (e.g. some mobile / data-saver contexts): keep the poster
+    playPromise.catch(() => video.remove());
   }
+}
 
-  if (videoEl) {
-    // Already a <video> element — use it directly
-    mediaSource = videoEl;
-    mediaEl = videoEl.cloneNode(true);
-    mediaEl.setAttribute('autoplay', '');
-    mediaEl.setAttribute('muted', '');
-    mediaEl.setAttribute('loop', '');
-    mediaEl.setAttribute('playsinline', '');
-    mediaEl.className = 'hero-media hero-video';
-  } else if (videoLink || plainVideoUrl) {
-    mediaSource = videoLink || firstRow;
-    const videoUrl = videoLink ? videoLink.href : plainVideoUrl;
+/**
+ * loads and decorates the hero
+ * @param {Element} block The hero block element
+ */
+export default function decorate(block) {
+  const isVideo = block.classList.contains('video');
 
-    if (videoUrl.match(/\.(mp4|webm)(\?|$)/i)) {
-      // Direct video file
-      const ext = videoUrl.match(/\.(mp4|webm)/i)[1];
-      mediaEl = document.createElement('video');
-      mediaEl.setAttribute('autoplay', '');
-      mediaEl.setAttribute('muted', '');
-      mediaEl.setAttribute('loop', '');
-      mediaEl.setAttribute('playsinline', '');
-      mediaEl.innerHTML = `<source src="${videoUrl}" type="video/${ext}">`;
-      mediaEl.className = 'hero-media hero-video';
-    } else if (videoUrl.match(/youtube|youtu\.be/i)) {
-      // YouTube embed
-      let embedUrl = videoUrl;
-      const ytMatch = videoUrl.match(/(?:youtube\.com\/watch\?v=|youtu\.be\/)([\w-]+)/);
-      if (ytMatch) {
-        embedUrl = `https://www.youtube.com/embed/${ytMatch[1]}?autoplay=1&mute=1&loop=1&controls=0&showinfo=0&playlist=${ytMatch[1]}`;
-      }
-      mediaEl = document.createElement('div');
-      mediaEl.className = 'hero-media hero-video hero-video-embed';
-      mediaEl.innerHTML = `<iframe src="${embedUrl}" frameborder="0" allow="autoplay; encrypted-media" allowfullscreen loading="lazy"></iframe>`;
-    } else if (videoUrl.match(/vimeo/i)) {
-      // Vimeo embed
-      const vimeoMatch = videoUrl.match(/vimeo\.com\/(\d+)/);
-      const vimeoId = vimeoMatch ? vimeoMatch[1] : '';
-      mediaEl = document.createElement('div');
-      mediaEl.className = 'hero-media hero-video hero-video-embed';
-      mediaEl.innerHTML = `<iframe src="https://player.vimeo.com/video/${vimeoId}?autoplay=1&muted=1&loop=1&background=1" frameborder="0" allow="autoplay" loading="lazy"></iframe>`;
-    }
-  } else if (picture) {
-    // Image fallback
-    mediaSource = picture;
+  // Poster / background image — this is the LCP element.
+  const picture = block.querySelector('picture');
+  let mediaEl;
+  if (picture) {
     mediaEl = picture.cloneNode(true);
     mediaEl.className = 'hero-media';
+    const img = mediaEl.querySelector('img');
+    if (img) {
+      img.setAttribute('loading', 'eager');
+      img.setAttribute('fetchpriority', 'high');
+    }
+  }
+
+  // Video source is honored only in the video variant.
+  let videoUrl = null;
+  let videoSource = null; // authored node to exclude from the text content
+  if (isVideo) {
+    const link = block.querySelector('a[href$=".mp4"], a[href$=".webm"]');
+    if (link) {
+      videoUrl = link.href;
+      videoSource = link;
+    } else {
+      const urlPara = [...block.querySelectorAll('p')]
+        .find((p) => VIDEO_URL_RE.test(p.textContent.trim()));
+      if (urlPara) {
+        videoUrl = urlPara.textContent.trim();
+        videoSource = urlPara;
+      }
+    }
   }
 
-  // Extract text content from all rows, excluding whatever became the media.
-  // Supports media in its own row (multi-row authoring) or sharing a row with
-  // the heading/text/CTA (single-row authoring).
+  // Collect the remaining content (heading, subtext, CTA), excluding the media.
   const contentEl = document.createElement('div');
   contentEl.className = 'hero-content';
-
-  rows.forEach((row) => {
+  [...block.children].forEach((row) => {
     const inner = row.querySelector(':scope > div') || row;
     [...inner.children].forEach((child) => {
-      if (mediaSource && (child === mediaSource || child.contains(mediaSource))) return;
+      if (picture && (child === picture || child.contains(picture))) return;
+      if (videoSource && (child === videoSource || child.contains(videoSource))) return;
       contentEl.append(child.cloneNode(true));
     });
   });
 
-  // Rebuild block
+  // Rebuild the block: poster image first, then overlaid content.
   block.textContent = '';
   if (mediaEl) block.append(mediaEl);
   block.append(contentEl);
+
+  // Defer the video until after LCP, and only when motion is allowed.
+  if (videoUrl && !window.matchMedia('(prefers-reduced-motion: reduce)').matches) {
+    const start = () => loadHeroVideo(block, videoUrl);
+    if (document.readyState === 'complete') {
+      start();
+    } else {
+      window.addEventListener('load', start, { once: true });
+    }
+  }
 }

From 80ba6b2c4026f5ed2672a0ae8a2d7eb6b5fe8405 Mon Sep 17 00:00:00 2001
From: Lamont Crook <lamont@adobe.com>
Date: Wed, 8 Jul 2026 14:01:50 -0600
Subject: [PATCH 2/4] Set first-section top margin to 38px

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 styles/styles.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/styles/styles.css b/styles/styles.css
index 617120e..898d83c 100644
--- a/styles/styles.css
+++ b/styles/styles.css
@@ -343,7 +343,7 @@ main > .section > div {
 }
 
 main > .section:first-of-type {
-  margin-top: 0;
+  margin-top: 38px;
 }
 
 @media (width >= 900px) {

From 16437f0f4925b09d94d70e9d00e9c494634aa9f4 Mon Sep 17 00:00:00 2001
From: Lamont Crook <lamont@adobe.com>
Date: Wed, 8 Jul 2026 14:06:25 -0600
Subject: [PATCH 3/4] Gate hero video on viewport/connection to protect mobile
 LCP

A full-bleed autoplay video overtakes the poster as the LCP element on slow
connections (PSI mobile LCP ~9.6s). Limit the video to >=900px viewports on
capable, non-data-saver connections; mobile and constrained clients keep the
eager poster image, which stays the LCP element (~fast). Desktop is unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 blocks/hero/hero.js | 21 +++++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/blocks/hero/hero.js b/blocks/hero/hero.js
index 1263b55..b2b7999 100644
--- a/blocks/hero/hero.js
+++ b/blocks/hero/hero.js
@@ -58,6 +58,23 @@ function loadHeroVideo(block, url) {
   }
 }
 
+/**
+ * Whether the video backdrop should load. It is a heavy autoplay asset, so it
+ * is limited to larger viewports on capable connections and skipped for
+ * reduced-motion or data-saver users. In those cases the poster image remains,
+ * which also keeps it as the LCP element on constrained (mobile) connections.
+ * @returns {boolean}
+ */
+function shouldLoadHeroVideo() {
+  if (window.matchMedia('(prefers-reduced-motion: reduce)').matches) return false;
+  if (!window.matchMedia('(min-width: 900px)').matches) return false;
+  const conn = navigator.connection;
+  if (conn && (conn.saveData || ['slow-2g', '2g', '3g'].includes(conn.effectiveType))) {
+    return false;
+  }
+  return true;
+}
+
 /**
  * loads and decorates the hero
  * @param {Element} block The hero block element
@@ -113,8 +130,8 @@ export default function decorate(block) {
   if (mediaEl) block.append(mediaEl);
   block.append(contentEl);
 
-  // Defer the video until after LCP, and only when motion is allowed.
-  if (videoUrl && !window.matchMedia('(prefers-reduced-motion: reduce)').matches) {
+  // Defer the video until after LCP, and only under capable conditions.
+  if (videoUrl && shouldLoadHeroVideo()) {
     const start = () => loadHeroVideo(block, videoUrl);
     if (document.readyState === 'complete') {
       start();

From 157622635691d71eceda28d9df93e2576e8b5e17 Mon Sep 17 00:00:00 2001
From: Lamont Crook <lamont@adobe.com>
Date: Wed, 8 Jul 2026 14:09:04 -0600
Subject: [PATCH 4/4] Re-run PSI

Empty commit to re-trigger the aem-psi-check after a transient desktop
Lighthouse timeout on the previous run.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>