From 54ca8615455c75a5c91c914db06b68d40b7ce4fb Mon Sep 17 00:00:00 2001 From: Jackson Harper Date: Thu, 15 Feb 2024 10:38:13 +0800 Subject: [PATCH] Reader header height should not take pinned searches into account This fixes an issue where the PDF toolbar can be covered by an "invisible" header element. --- packages/web/components/templates/reader/ReaderHeader.tsx | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/packages/web/components/templates/reader/ReaderHeader.tsx b/packages/web/components/templates/reader/ReaderHeader.tsx index 76287962a..5c30294e7 100644 --- a/packages/web/components/templates/reader/ReaderHeader.tsx +++ b/packages/web/components/templates/reader/ReaderHeader.tsx @@ -3,7 +3,10 @@ import { Button } from '../../elements/Button' import { PrimaryDropdown } from '../PrimaryDropdown' import { LogoBox } from '../../elements/LogoBox' import { ReactNode } from 'react' -import { useGetHeaderHeight } from '../homeFeed/HeaderSpacer' +import { + DEFAULT_HEADER_HEIGHT, + useGetHeaderHeight, +} from '../homeFeed/HeaderSpacer' import { theme } from '../../tokens/stitches.config' import { ReaderSettingsIcon } from '../../elements/icons/ReaderSettingsIcon' import { CircleUtilityMenuIcon } from '../../elements/icons/CircleUtilityMenuIcon' @@ -16,7 +19,6 @@ type ReaderHeaderProps = { } export function ReaderHeader(props: ReaderHeaderProps): JSX.Element { - const headerHeight = useGetHeaderHeight() return ( <>