Files
Joshua Ryder 3f1c573a67 feat: Make menu context-aware and hide current page links
Implement intelligent menu filtering that hides self-referencing links and shows all applicable pages based on user role and authentication state.

Changes:
- Menu component now uses currentPath to filter out the current page link
- Added computed properties to detect which page user is on
- Fixed profile, admin, and users pages to dynamically detect admin status from API
- Added menu to forgot-password page for consistent navigation
- All pages now pass correct authentication state to Menu component

This ensures menus always show relevant navigation options while avoiding redundant links to the current page. Admin users now see all admin options (Manage Sermons, Manage Users) regardless of which page they're on, except the current one.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-07 09:40:42 -05:00

197 lines
6.1 KiB
Vue

<template>
<div class="relative" ref="menuContainer">
<!-- Hamburger Button -->
<button
@click="toggleMenu"
:disabled="isToggling"
class="p-2 text-gray-700 hover:bg-gray-100 rounded-md focus:outline-none focus:ring-2 focus:ring-blue-500 disabled:opacity-50"
aria-label="Menu"
>
<svg class="w-6 h-6" fill="none" stroke="currentColor" viewBox="0 0 24 24">
<path
v-if="!isOpen"
stroke-linecap="round"
stroke-linejoin="round"
stroke-width="2"
d="M4 6h16M4 12h16M4 18h16"
/>
<path
v-else
stroke-linecap="round"
stroke-linejoin="round"
stroke-width="2"
d="M6 18L18 6M6 6l12 12"
/>
</svg>
</button>
<!-- Dropdown Menu -->
<Transition
enter-active-class="transition ease-out duration-100"
enter-from-class="transform opacity-0 scale-95"
enter-to-class="transform opacity-100 scale-100"
leave-active-class="transition ease-in duration-75"
leave-from-class="transform opacity-100 scale-100"
leave-to-class="transform opacity-0 scale-95"
>
<div
v-if="isOpen"
class="absolute right-0 mt-2 w-56 bg-white rounded-md shadow-lg ring-1 ring-black ring-opacity-5 z-50"
>
<div class="py-1">
<!-- Home Link -->
<NuxtLink
v-if="showHome && !isOnHomePage"
to="/"
@click="isOpen = false"
class="block px-4 py-2 text-sm text-gray-700 hover:bg-gray-100"
>
🏠 Home
</NuxtLink>
<!-- Authentication Links -->
<ClientOnly>
<template v-if="isAuthenticated">
<!-- Profile Link -->
<NuxtLink
v-if="!isOnProfilePage"
to="/profile"
@click="isOpen = false"
class="block px-4 py-2 text-sm text-gray-700 hover:bg-gray-100"
>
👤 Edit Profile
</NuxtLink>
<!-- Admin Links -->
<template v-if="isAdmin">
<NuxtLink
v-if="!isOnAdminPage"
to="/admin"
@click="isOpen = false"
class="block px-4 py-2 text-sm text-gray-700 hover:bg-gray-100"
>
📝 Manage Sermons
</NuxtLink>
<NuxtLink
v-if="!isOnUsersPage"
to="/users"
@click="isOpen = false"
class="block px-4 py-2 text-sm text-gray-700 hover:bg-gray-100"
>
👥 Manage Users
</NuxtLink>
</template>
<!-- Logout -->
<button
@click="handleLogout"
class="block w-full text-left px-4 py-2 text-sm text-red-700 hover:bg-red-50"
>
🚪 Log Out
</button>
</template>
<!-- Not authenticated -->
<template v-else>
<NuxtLink
v-if="!isOnLoginPage"
:to="loginRedirectUrl"
@click="isOpen = false"
class="block px-4 py-2 text-sm text-green-700 hover:bg-green-50 font-medium"
>
🔑 Log In
</NuxtLink>
<NuxtLink
v-if="!isOnRegisterPage"
to="/register"
@click="isOpen = false"
class="block px-4 py-2 text-sm text-blue-700 hover:bg-blue-50 font-medium"
>
Register
</NuxtLink>
</template>
</ClientOnly>
</div>
</div>
</Transition>
</div>
</template>
<script setup lang="ts">
const props = defineProps<{
isAuthenticated?: boolean
isAdmin?: boolean
showHome?: boolean
currentPath?: string
}>()
const isOpen = ref(false)
const isToggling = ref(false)
const menuContainer = ref<HTMLElement | null>(null)
const route = useRoute()
const loginRedirectUrl = computed(() => {
const redirectPath = props.currentPath || route.fullPath
return `/login?redirect=${encodeURIComponent(redirectPath)}`
})
// Determine which page we're currently on to hide self-referencing links
const currentPathNormalized = computed(() => {
const path = props.currentPath || route.fullPath
// Remove query params and trailing slashes for comparison
return path.split('?')[0].replace(/\/$/, '')
})
const isOnHomePage = computed(() => currentPathNormalized.value === '' || currentPathNormalized.value === '/')
const isOnProfilePage = computed(() => currentPathNormalized.value === '/profile')
const isOnAdminPage = computed(() => currentPathNormalized.value === '/admin')
const isOnUsersPage = computed(() => currentPathNormalized.value === '/users')
const isOnLoginPage = computed(() => currentPathNormalized.value === '/login')
const isOnRegisterPage = computed(() => currentPathNormalized.value === '/register')
// Toggle menu with debounce to prevent accidental double-taps
const toggleMenu = () => {
if (isToggling.value) return
isToggling.value = true
isOpen.value = !isOpen.value
// Re-enable after 500ms
setTimeout(() => {
isToggling.value = false
}, 500)
}
const handleLogout = async () => {
isOpen.value = false
const currentPath = props.currentPath || route.fullPath
await $fetch('/api/auth/logout', { method: 'POST' })
// Force a full page reload to refresh authentication state
// This ensures all components update correctly
if (import.meta.client) {
window.location.href = currentPath
}
}
// Close menu when clicking outside
const handleClickOutside = (event: MouseEvent) => {
if (!isOpen.value || !menuContainer.value) return
const target = event.target as HTMLElement
// Check if click is outside the menu container
if (!menuContainer.value.contains(target)) {
isOpen.value = false
}
}
onMounted(() => {
// Use capture phase to ensure we catch the event before other handlers
document.addEventListener('click', handleClickOutside, true)
})
onUnmounted(() => {
document.removeEventListener('click', handleClickOutside, true)
})
</script>