Skip to content

Commit 4a4744c

Browse files
committed
Fix: Ensure main navigation menu is always visible and functional
- Add explicit display styles to nav-menu-item class - Ensure desktop menu is visible on large screens with proper flex layout - Add flex-1 to menu container for proper spacing - Add relative positioning to menu container - Replace @apply with explicit CSS for nav-menu-item to ensure visibility - Fix menu visibility issues that prevented navigation from appearing
1 parent c1414c2 commit 4a4744c

File tree

2 files changed

+24
-7
lines changed

2 files changed

+24
-7
lines changed

_includes/portal.html

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,8 @@
9696
</div>
9797

9898
<!-- Main Menu Bar -->
99-
<div class="container-padding">
100-
<div class="flex items-center justify-between">
99+
<div class="container-padding relative">
100+
<div class="flex items-center justify-between w-full">
101101
<!-- Mobile menu button -->
102102
<button
103103
class="lg:hidden py-3 px-2 text-white hover:bg-stone-700 focus:outline-none focus:ring-2 focus:ring-orange-500 rounded-md transition-colors"
@@ -111,8 +111,8 @@
111111
</svg>
112112
</button>
113113

114-
<!-- Desktop Navigation -->
115-
<ul class="hidden lg:flex flex-row items-center" role="menubar" id="main-menu-desktop">
114+
<!-- Desktop Navigation - Always visible on large screens -->
115+
<ul class="hidden lg:flex flex-row items-center flex-1" role="menubar" id="main-menu-desktop">
116116
<li role="none">
117117
<a class="nav-menu-item" href="/development" role="menuitem" aria-label="Development">DEVELOPMENT</a>
118118
</li>

_sass/main.scss

Lines changed: 20 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,26 @@ select:focus {
6565
}
6666

6767
.nav-menu-item {
68-
@apply flex items-center justify-center h-16 px-4 font-semibold text-white;
69-
@apply transition-colors hover:bg-stone-700;
70-
@apply focus:outline-none focus:ring-2 focus:ring-inset focus:ring-orange-500;
68+
display: flex !important;
69+
align-items: center;
70+
justify-content: center;
71+
height: 4rem;
72+
padding: 0 1rem;
73+
font-weight: 600;
74+
color: #ffffff;
75+
transition-property: background-color;
76+
transition-timing-function: cubic-bezier(0.4, 0, 0.2, 1);
77+
transition-duration: 150ms;
78+
}
79+
80+
.nav-menu-item:hover {
81+
background-color: #57534e;
82+
}
83+
84+
.nav-menu-item:focus {
85+
outline: 2px solid transparent;
86+
outline-offset: 2px;
87+
box-shadow: inset 0 0 0 2px #f97316;
7188
}
7289

7390
.nav-menu-item-mobile {

0 commit comments

Comments
 (0)