Skip to content

Commit fb287bf

Browse files
committed
fix: prevent LogoContextMenu wrapper from breaking mobile nav layout
Use display:contents so the wrapper div doesn't participate in flex layout.
1 parent eeef5e5 commit fb287bf

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

app/components/AppHeader.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -226,11 +226,11 @@ onKeyStroke(
226226
class="relative container min-h-14 flex items-center gap-2 z-1 justify-end"
227227
>
228228
<!-- Mobile: Logo (navigates home) -->
229-
<LogoContextMenu v-if="!isSearchExpanded && !isOnHomePage" class="sm:hidden">
229+
<LogoContextMenu v-if="!isSearchExpanded && !isOnHomePage" class="sm:hidden flex-shrink-0">
230230
<NuxtLink
231231
to="/"
232232
:aria-label="$t('header.home')"
233-
class="flex-shrink-0 font-mono text-lg font-medium text-fg hover:text-fg transition-colors duration-200 focus-ring me-4"
233+
class="font-mono text-lg font-medium text-fg hover:text-fg transition-colors duration-200 focus-ring me-4"
234234
>
235235
<AppMark class="w-6 h-auto" />
236236
</NuxtLink>

app/components/LogoContextMenu.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ onKeyStroke('Escape', () => {
5656
</script>
5757

5858
<template>
59-
<div @contextmenu="onContextMenu">
59+
<div class="contents" @contextmenu="onContextMenu">
6060
<slot />
6161

6262
<Teleport to="body">

0 commit comments

Comments
 (0)