starred/webmail
1
0
Fork 0
mirror of https://github.com/bulwarkmail/webmail.git synced 2026-10-31 00:12:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #948 from shukiv/fix/batch-toolbar-top-of-list

Browse Source 
fix(list): stop the batch toolbar hiding the message you just selected
This commit is contained in:
Linus Rath
2026-09-01 20:18:28 +02:00
committed by GitHub
parent 3f2837277c 71dd5f4fed
commit f2415d266c
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
components/email/email-list.tsx
+7 -1
View File
@@ -156,6 +156,11 @@ export function EmailList({
// open it shrinks the list and would shove every row downwards. Feed each
// height change back into scrollTop so the rows stay put on screen (and
// slide back when the toolbar collapses again).
//
// Except at the very top: there are no rows above to hold steady, so the
// compensation just scrolls the first message underneath the toolbar and
// reads as the toolbar covering the message you selected. Let the list
// move down there instead.
useEffect(() => {
const toolbar = batchToolbarRef.current;
if (!toolbar || typeof ResizeObserver === 'undefined') return;
@@ -166,7 +171,8 @@ export function EmailList({
lastHeight = height;
const list = parentRef.current;
if (!list || delta === 0) return;
list.scrollTop += delta;
if (delta > 0 && list.scrollTop <= 0) return;
list.scrollTop = Math.max(0, list.scrollTop + delta);
});
observer.observe(toolbar);
return () => observer.disconnect();