From f61ba506a961641a9c812e17774992d4b713475a Mon Sep 17 00:00:00 2001
From: Jon Froehlich <jonf@cs.uw.edu>
Date: Mon, 15 Jun 2026 09:23:47 -0700
Subject: [PATCH] Remove redundant allowfullscreen on video iframes (#1296)

The video-embed iframes set both allow="autoplay; fullscreen" and the legacy
allowfullscreen attribute; Chrome warns that the allow Permissions-Policy
token takes precedence. The allow token already grants fullscreen, so
allowfullscreen is redundant. Removing it silences the console warning with no
behavior change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 website/templates/snippets/display_video_snippet.html | 3 +--
 website/templates/website/project.html                | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/website/templates/snippets/display_video_snippet.html b/website/templates/snippets/display_video_snippet.html
index 07f084af..c0b76458 100644
--- a/website/templates/snippets/display_video_snippet.html
+++ b/website/templates/snippets/display_video_snippet.html
@@ -47,8 +47,7 @@
             src="{{ video.get_embed }}" 
             title="Video: {{ video.title }}"
             frameborder="0" 
-            allow="autoplay; fullscreen" 
-            allowfullscreen
+            allow="autoplay; fullscreen"
             loading="lazy">
     </iframe>
   </div>
diff --git a/website/templates/website/project.html b/website/templates/website/project.html
index ed06b75d..f30e2365 100644
--- a/website/templates/website/project.html
+++ b/website/templates/website/project.html
@@ -159,8 +159,7 @@ <h1 class="sr-only">{{ project.name }}</h1>
           <iframe class="project-featured-video-player" 
                   src="{{ featured_video.get_embed }}" 
                   title="Featured video: {{ featured_video.title }}"
-                  allow="autoplay; fullscreen" 
-                  allowfullscreen
+                  allow="autoplay; fullscreen"
                   loading="lazy">
           </iframe>
         </div>