From 1c377fdce64c1dbe1c2c20e0bc3d1bebc5e13a8e Mon Sep 17 00:00:00 2001
From: Israel Cefrin <israel.cefrin@gmail.com>
Date: Thu, 11 Jan 2024 04:10:00 +0000
Subject: [PATCH] pkp/pkp-lib#5199 Set the current page on the top menu  - OJS
 3.4

---
 .../frontend/components/navigationMenu.tpl    | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/templates/frontend/components/navigationMenu.tpl b/templates/frontend/components/navigationMenu.tpl
index 6f516f40308..d178c521b7e 100644
--- a/templates/frontend/components/navigationMenu.tpl
+++ b/templates/frontend/components/navigationMenu.tpl
@@ -19,16 +19,27 @@
 			{if !$navigationMenuItemAssignment->navigationMenuItem->getIsDisplayed()}
 				{continue}
 			{/if}
-			<li class="{$liClass|escape}">
-				<a href="{$navigationMenuItemAssignment->navigationMenuItem->getUrl()}">
+
+			{* Check if menu item has submenu *}
+			{if $navigationMenuItemAssignment->navigationMenuItem->getIsChildVisible()}
+				{assign var=hasSubmenu value=true}
+			{else}
+				{assign var=hasSubmenu value=false}
+			{/if}
+			{* Check if is current page for aria-current parameter*}
+			{assign var="isActive" value={default_item_active item=$navigationMenuItemAssignment->navigationMenuItem}}
+
+		<li class="{$liClass|escape} main-menu__nav-item{if $hasSubmenu} dropdown{/if} {default_item_active item=$navigationMenuItemAssignment->navigationMenuItem}" >
+				<a {if $isActive} aria-current="page" {/if}{if $hasSubmenu} aria-current="true"{/if} href="{$navigationMenuItemAssignment->navigationMenuItem->getUrl()}">
 					{$navigationMenuItemAssignment->navigationMenuItem->getLocalizedTitle()}
 				</a>
 				{if $navigationMenuItemAssignment->navigationMenuItem->getIsChildVisible()}
 					<ul>
 						{foreach key=childField item=childNavigationMenuItemAssignment from=$navigationMenuItemAssignment->children}
+							{assign var="isActive2" value={default_item_active item=$childNavigationMenuItemAssignment->navigationMenuItem}}
 							{if $childNavigationMenuItemAssignment->navigationMenuItem->getIsDisplayed()}
-								<li class="{$liClass|escape}">
-									<a href="{$childNavigationMenuItemAssignment->navigationMenuItem->getUrl()}">
+								<li class="{$liClass|escape} {default_item_active item=$childNavigationMenuItemAssignment->navigationMenuItem}">
+									<a href="{$childNavigationMenuItemAssignment->navigationMenuItem->getUrl()}" {if $isActive2} aria-current="page" {/if} >
 										{$childNavigationMenuItemAssignment->navigationMenuItem->getLocalizedTitle()}
 									</a>
 								</li>