Salastil/odysseus
1
0
Fork 0
mirror of https://github.com/pewdiepie-archdaemon/odysseus.git synced 2026-06-17 10:15:27 -04:00
Code Issues Packages Projects Releases Wiki Activity

Cookbook auto-fold: target the actual scroll container (.cookbook-body)

Browse Source 
Previous .modal-body / .cookbook-content lookup matched neither the
desktop scroller (.cookbook-body) nor the mobile one (#cookbook-modal
.modal-content), so the scroll listener was attached to document.body
and never fired. Walk up to whichever scroller actually exists.
This commit is contained in:
pewdiepie-archdaemon
2026-06-13 20:05:33 +09:00
parent d68c75a82c
commit ae0b29af3d
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
static/js/cookbook.js
+9 -6
View File
@@ -1416,17 +1416,20 @@ function _wireTabEvents(body) {
_setFolded(!folded);
});
// Auto-fold when the user scrolls past the Direct Download header.
// Watches the header element: once its bottom edge passes above the
// scroll container's top, fold the body so the scan section below has
// the full viewport. Doesn't auto-unfold (user can click ▸ to expand).
const _scrollHost = dlFold.closest('.modal-body, .cookbook-content, .doclib-modal-content') || document.scrollingElement || document.body;
let _autoFolded = false;
// Desktop scroll container is .cookbook-body; mobile is the modal
// .modal-content. Walk up via .closest() to find whichever is
// actually scrollable. Doesn't auto-unfold — chevron ▸ still
// expands manually.
const _scrollHost = dlFold.closest('.cookbook-body')
|| dlFold.closest('#cookbook-modal .modal-content')
|| dlFold.closest('.modal-content')
|| document.scrollingElement
|| document.body;
const _onScroll = () => {
if (dlFoldBody.style.display === 'none') return;
const r = dlFold.getBoundingClientRect();
const top = (_scrollHost && _scrollHost.getBoundingClientRect) ? _scrollHost.getBoundingClientRect().top : 0;
if (r.bottom < top + 4) {
_autoFolded = true;
_setFolded(true, /* persist */ false);
}
};