Use flex sizing for snapshot previews

This commit is contained in:
Nick Sweeting 2026-08-12 16:25:51 -07:00
parent ebd617677e
commit 3bb99db2f9
No known key found for this signature in database
2 changed files with 6 additions and 11 deletions

View File

@ -833,11 +833,12 @@
}
#main-frame-wrapper {
width: 100%;
height: calc(100vh - 210px);
height: auto;
border-top: 3px solid #aa1e55;
overflow: hidden;
order: 2;
flex: 0 0 auto;
flex: 1 1 auto;
min-height: 0;
display: flex;
flex-direction: column;
}
@ -857,7 +858,7 @@
}
.full-page-wrapper {
width: 100%;
height: calc(100vh - 210px);
min-height: 0;
}
.snapshot-empty-state {
display: flex;
@ -1027,9 +1028,6 @@
height: 32px;
max-height: 32px;
}
#main-frame-wrapper.iframe-large {
height: calc(100vh - 70px);
}
.preview-hidden {
display: none !important;
}
@ -2094,7 +2092,6 @@
console.log('Collapsing Snapshot header...')
jQuery('.header-toggle').text('▸')
jQuery('.header-bottom').hide()
jQuery('#main-frame-wrapper').addClass('iframe-large')
try {
localStorage.setItem("archivebox-snapshot-header-visible", "false")
} catch (e) {
@ -2105,7 +2102,6 @@
console.log('Expanding Snapshot header...')
jQuery('.header-toggle').text('▾')
jQuery('.header-bottom').show()
jQuery('#main-frame-wrapper').removeClass('iframe-large')
try {
localStorage.setItem("archivebox-snapshot-header-visible", "true")
} catch (e) {

View File

@ -694,9 +694,8 @@ 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
assert "height: calc(100vh - 210px)" not in rendered
assert "iframe-large" not in rendered
class TestAdminSnapshotListView: