From 183775bf32edce010055dfd56b6c579497f27150 Mon Sep 17 00:00:00 2001
From: Nick Sweeting <git@sweeting.me>
Date: Wed, 12 Aug 2026 11:02:23 -0700
Subject: [PATCH] Resize snapshot previews with header state

---
 archivebox/templates/core/snapshot.html    | 11 ++++-------
 archivebox/tests/test_ui_admin_snapshot.py |  3 +++
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/archivebox/templates/core/snapshot.html b/archivebox/templates/core/snapshot.html
index b84cb7c8..a982a792 100644
--- a/archivebox/templates/core/snapshot.html
+++ b/archivebox/templates/core/snapshot.html
@@ -826,7 +826,7 @@
             #main-frame {
                 border-top: 1px solid #ddd;
                 width: 100%;
-                height: calc(100vh - 210px);
+                height: 100%;
                 margin: 0px;
                 border: 0px;
                 border-top: 3px solid #aa1e55;
@@ -1027,7 +1027,7 @@
                 height: 32px;
                 max-height: 32px;
             }
-            .iframe-large {
+            #main-frame-wrapper.iframe-large {
                 height: calc(100vh - 70px);
             }
             .preview-hidden {
@@ -1948,9 +1948,6 @@
                 frame.sandbox = "allow-same-origin allow-top-navigation-by-user-activation allow-scripts allow-forms"
                 frame.loading = 'eager'
                 frame.setAttribute('fetchpriority', 'high')
-                if (previousFrame && previousFrame.classList.contains('iframe-large')) {
-                    frame.classList.add('iframe-large')
-                }
                 attachPreviewFrameHandlers(frame)
                 return frame
             }
@@ -2097,7 +2094,7 @@
                 console.log('Collapsing Snapshot header...')
                 jQuery('.header-toggle').text('▸')
                 jQuery('.header-bottom').hide()
-                jQuery('#main-frame').addClass('iframe-large')
+                jQuery('#main-frame-wrapper').addClass('iframe-large')
                 try {
                     localStorage.setItem("archivebox-snapshot-header-visible", "false")
                 } catch (e) {
@@ -2108,7 +2105,7 @@
                 console.log('Expanding Snapshot header...')
                 jQuery('.header-toggle').text('▾')
                 jQuery('.header-bottom').show()
-                jQuery('#main-frame').removeClass('iframe-large')
+                jQuery('#main-frame-wrapper').removeClass('iframe-large')
                 try {
                     localStorage.setItem("archivebox-snapshot-header-visible", "true")
                 } catch (e) {
diff --git a/archivebox/tests/test_ui_admin_snapshot.py b/archivebox/tests/test_ui_admin_snapshot.py
index 7d9e6c5e..3d0dcde2 100644
--- a/archivebox/tests/test_ui_admin_snapshot.py
+++ b/archivebox/tests/test_ui_admin_snapshot.py
@@ -694,6 +694,9 @@ class TestSnapshotProgressStats:
 
         assert "full-page-iframe" in rendered
         assert "index.jsonl?preview=1" in rendered
+        assert "#main-frame-wrapper.iframe-large" in rendered
+        assert "jQuery('#main-frame-wrapper').addClass('iframe-large')" in rendered
+        assert "jQuery('#main-frame-wrapper').removeClass('iframe-large')" in rendered
 
 
 class TestAdminSnapshotListView: