Skip to content

Commit 090950a

Browse files
committed
fix: reduce mobile banner height to 120px to remove gap
1 parent 9499eb3 commit 090950a

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

src/components/LanguageSuggestionBanner.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@ const LANGUAGE_PREFERENCE_KEY = 'folio-language-preference'
99

1010
// Banner height in pixels (measured from Figma)
1111
// Desktop: h-[68px] as per Figma (py-4 + single row)
12-
// Mobile: py-6 (48px) + text row (~60px with wrap) + gap (16px) + buttons (36px) = ~160px
12+
// Mobile: py-6 (48px) + text row (~20px) + gap (16px) + buttons (36px) = ~120px
1313
export const BANNER_HEIGHT_DESKTOP = 68
14-
export const BANNER_HEIGHT_MOBILE = 160
14+
export const BANNER_HEIGHT_MOBILE = 120
1515

1616
/**
1717
* Spacer component that reserves space for the fixed banner.
@@ -24,7 +24,7 @@ export function BannerSpacer() {
2424
return null
2525
}
2626

27-
return <div className="h-[160px] md:h-[68px]" aria-hidden="true" />
27+
return <div className="h-[120px] md:h-[68px]" aria-hidden="true" />
2828
}
2929

3030
/**

src/components/Navbar.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,9 @@ export default function Navbar() {
3737
useBodyScrollLock(isMobileMenuOpen)
3838

3939
// Navbar position: below banner when visible
40-
// Mobile banner: ~160px (with text wrap), Desktop banner: 68px (from Figma)
40+
// Mobile banner: ~120px, Desktop banner: 68px (from Figma)
4141
const topOffset = isBannerVisible
42-
? 'top-[160px] md:top-[68px]'
42+
? 'top-[120px] md:top-[68px]'
4343
: 'top-0'
4444

4545
return (

src/components/navbar/MobileMenu.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,9 +124,9 @@ const PLATFORM_ITEMS = [
124124
]
125125

126126
// Navbar height (64px) + banner height when visible
127-
// Mobile banner: ~160px (with text wrap per Figma design)
127+
// Mobile banner: ~120px per Figma design
128128
const NAVBAR_HEIGHT = 64
129-
const BANNER_HEIGHT_MOBILE = 160
129+
const BANNER_HEIGHT_MOBILE = 120
130130

131131
export default function MobileMenu({ isOpen, onClose, bannerVisible = false }: MobileMenuProps) {
132132
const { t } = useTranslation('common')

0 commit comments

Comments
 (0)