Skip to content

Commit 1e3b2aa

Browse files
ferranregoclaude
andauthored
fix(navbar): align mobile sidebar typography for Solutions and Resources (#67)
Co-authored-by: Ferran <ferran@vocdoni.org> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 9f8abc7 commit 1e3b2aa

2 files changed

Lines changed: 15 additions & 11 deletions

File tree

components/Link.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ const linkVariants = cva('transition-colors', {
1313
navbarItem:
1414
'block select-none space-y-1 rounded-md p-3 leading-none no-underline outline-none transition-colors hover:bg-accent hover:text-accent-foreground focus:bg-accent focus:text-accent-foreground',
1515
navbarMobile: 'text-sm text-muted-foreground hover:text-foreground py-2 block',
16-
navbarStatic: 'text-sm font-medium hover:text-primary transition-colors py-2 border-b border-border/40 block',
16+
navbarStatic: 'text-sm font-medium hover:text-primary transition-colors py-3 border-b border-border/40 block',
1717
footerNav: 'text-sm text-muted-foreground hover:text-foreground transition-colors',
1818
footerLegal: 'text-muted-foreground hover:text-foreground transition-colors',
1919
heroBadge:

components/Navbar.tsx

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -106,13 +106,13 @@ export function Navbar() {
106106
{/* Logo */}
107107
<div className='pointer-events'>
108108
<Link href='/' variant='unstyled' aria-label={t('navbar.logo_aria_label', 'Vocdoni - go to homepage')}>
109-
<VocdoniLogo minimal className='h-7 lg:hidden' aria-hidden='true' />
110-
<VocdoniLogo className='hidden lg:block h-8' aria-hidden='true' />
109+
<VocdoniLogo minimal className='h-7 xl:hidden' aria-hidden='true' />
110+
<VocdoniLogo className='hidden xl:block h-8' aria-hidden='true' />
111111
</Link>
112112
</div>
113113

114114
{/* Desktop Navigation */}
115-
<NavigationMenu className='hidden lg:flex min-w-max'>
115+
<NavigationMenu className='hidden xl:flex min-w-max'>
116116
<NavigationMenuList>
117117
{/* Solutions (formerly Product) */}
118118
<NavigationMenuItem>
@@ -243,7 +243,7 @@ export function Navbar() {
243243
<div className='flex items-center gap-2'>
244244
<LanguageSwitcher />
245245
{/* Mobile Menu Trigger */}
246-
<div className='lg:hidden'>
246+
<div className='xl:hidden'>
247247
<Sheet open={isOpen} onOpenChange={setIsOpen}>
248248
<SheetTrigger asChild>
249249
<Button variant='ghost' size='icon'>
@@ -262,8 +262,10 @@ export function Navbar() {
262262
<div className='flex-1 overflow-auto py-6 px-4'>
263263
<Accordion type='single' collapsible className='w-full'>
264264
{/* Solutions (mobile) */}
265-
<AccordionItem value='product'>
266-
<AccordionTrigger className='text-sm font-medium'>{t('navbar.solutions')}</AccordionTrigger>
265+
<AccordionItem value='product' className='border-border/40'>
266+
<AccordionTrigger className='font-sans text-sm font-medium py-3 hover:text-primary hover:no-underline transition-colors'>
267+
{t('navbar.solutions')}
268+
</AccordionTrigger>
267269
<AccordionContent>
268270
<div className='flex flex-col space-y-2 pl-4'>
269271
{/* Featured solution - mobile version */}
@@ -324,8 +326,10 @@ export function Navbar() {
324326
</AccordionItem>
325327

326328
{/* Resources (mobile) */}
327-
<AccordionItem value='resources'>
328-
<AccordionTrigger className='text-sm font-medium'>{t('navbar.resources')}</AccordionTrigger>
329+
<AccordionItem value='resources' className='border-border/40'>
330+
<AccordionTrigger className='font-sans text-sm font-medium py-3 hover:text-primary hover:no-underline transition-colors'>
331+
{t('navbar.resources')}
332+
</AccordionTrigger>
329333
<AccordionContent>
330334
<div className='flex flex-col space-y-2 pl-4'>
331335
{resourcesItems.map((item) => (
@@ -346,7 +350,7 @@ export function Navbar() {
346350
</Accordion>
347351

348352
{/* Static Links */}
349-
<div className='mt-4 flex flex-col space-y-4'>
353+
<div className='flex flex-col'>
350354
<Link href='/use-cases' variant='navbarStatic' onClick={() => setIsOpen(false)}>
351355
{t('navbar.use_cases')}
352356
</Link>
@@ -388,7 +392,7 @@ export function Navbar() {
388392
</div>
389393

390394
{/* App Button (hidden on mobile) */}
391-
<div className='hidden lg:block'>
395+
<div className='hidden xl:block'>
392396
<Button asChild className='rounded-full px-6'>
393397
<Link href='https://app.vocdoni.io' target='_blank' rel='noopener noreferrer' variant='unstyled'>
394398
{t('navbar.app_button')}

0 commit comments

Comments
 (0)