Skip to content

Commit 8c40b16

Browse files
Eranclaude
andcommitted
fix: close scroll container before response panel (was missing closing div)
The response panel was incorrectly nested inside the scroll container, causing it to scroll with content and leaving blank space on short tabs (Scheduled, Profile Settings). Added missing </div> to seal the scroll container, making the response panel a proper sticky footer. Also: added id="content-scroll" for reliable scroll-reset targeting, and updated selector from main > .border-t to main > div.border-t. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 55daed3 commit 8c40b16

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

index.html

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ <h2 class="text-base font-bold text-gray-900">Connection Settings</h2>
224224

225225
<!-- Main -->
226226
<main class="flex-1 flex flex-col overflow-hidden">
227-
<div class="flex-1 overflow-y-auto p-6">
227+
<div id="content-scroll" class="flex-1 overflow-y-auto p-6">
228228

229229
<!-- COMPOSE -->
230230
<div id="tab-text" class="tab-panel active max-w-2xl">
@@ -1098,6 +1098,8 @@ <h3 class="text-sm font-semibold text-gray-800">Privacy Settings</h3>
10981098
</div>
10991099
</div>
11001100

1101+
</div><!-- end scroll container -->
1102+
11011103
<!-- Response Panel -->
11021104
<div class="border-t border-gray-200 bg-white">
11031105
<div class="flex items-center justify-between px-6 py-3 border-b border-gray-100">
@@ -1747,11 +1749,11 @@ <h3 class="text-sm font-semibold text-gray-800">Privacy Settings</h3>
17471749
btn.classList.remove('text-gray-600');
17481750
clearResponse();
17491751
// Reset scroll to top
1750-
const scrollArea = document.querySelector('main > .overflow-y-auto');
1752+
const scrollArea = document.getElementById('content-scroll');
17511753
if (scrollArea) scrollArea.scrollTop = 0;
17521754
// Hide the response panel for tabs that don't make direct API calls
17531755
const noResponsePanel = ['scheduled', 'settings'];
1754-
const panel = document.querySelector('main > .border-t');
1756+
const panel = document.querySelector('main > div.border-t');
17551757
if (panel) panel.style.display = noResponsePanel.includes(name) ? 'none' : '';
17561758
}
17571759

0 commit comments

Comments
 (0)