ChoChoX/suwayomi-material-you-webui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Properly reset "app bar height"

Browse Source
This commit is contained in:
schroda
2024-10-24 17:30:24 +02:00
parent e416a141b0
commit 050fbc794c
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/modules/navigation-bar/components/DefaultNavBar.tsx
View File

@@ -6,7 +6,7 @@
* file, You can obtain one at https://mozilla.org/MPL/2.0/. * file, You can obtain one at https://mozilla.org/MPL/2.0/.
*/ */
import { useCallback, useContext, useEffect, useLayoutEffect, useMemo, useRef } from 'react'; import { useCallback, useContext, useLayoutEffect, useMemo, useRef } from 'react';
import AppBar from '@mui/material/AppBar'; import AppBar from '@mui/material/AppBar';
import Toolbar from '@mui/material/Toolbar'; import Toolbar from '@mui/material/Toolbar';
import Typography from '@mui/material/Typography'; import Typography from '@mui/material/Typography';
@@ -92,7 +92,13 @@ export function DefaultNavBar() {
appBarRef, appBarRef,
useCallback(() => setAppBarHeight(appBarRef.current?.clientHeight ?? 0), [appBarRef.current]), useCallback(() => setAppBarHeight(appBarRef.current?.clientHeight ?? 0), [appBarRef.current]),
); );
useEffect(() => setAppBarHeight(0), []); useLayoutEffect(() => {
if (!override.status) {
setAppBarHeight(0);
}
return () => setAppBarHeight(0);
}, [override.status]);
const activeNavBar: NavbarItem['show'] = isMobileWidth ? 'mobile' : 'desktop'; const activeNavBar: NavbarItem['show'] = isMobileWidth ? 'mobile' : 'desktop';
const visibleNavBarItems = useMemo( const visibleNavBarItems = useMemo(