From b2edd00fa8ad99b8492adc2327505729c76efed2 Mon Sep 17 00:00:00 2001
From: JT Smith <jt@plainblack.com>
Date: Thu, 22 Aug 2002 03:28:48 +0000
Subject: [PATCH] Fixed a bug where every navigation element was marked as the
 selected menu item.

---
 lib/WebGUI/Navigation.pm | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/lib/WebGUI/Navigation.pm b/lib/WebGUI/Navigation.pm
index db51c0d5a..f5d68a8a7 100644
--- a/lib/WebGUI/Navigation.pm
+++ b/lib/WebGUI/Navigation.pm
@@ -79,9 +79,13 @@ sub drawHorizontal {
 		} else {
 			$output .= ' '.$seperator.' ';
 		}
-                $output .= '<a class="'.$class.'" href="'.$tree->{$pageId}{url}
-                        .'"><span class="selectedMenuItem">'
-                        .$tree->{$pageId}{title}.'</span></a>';
+                $output .= '<a class="'.$class.'" href="'.$tree->{$pageId}{url}.'">';
+		if ($pageId == $session{page}{pageId}) {
+                       $output .= '<span class="selectedMenuItem">'.$tree->{$pageId}{title}.'</span>';
+		} else {
+                       $output .= $tree->{$pageId}{title};
+               	}
+                $output .= '</a>';
         }
         return $output;
 }
@@ -128,9 +132,13 @@ sub drawVertical {
                 $leading .= "<br>";
         }
 	foreach $pageId (keys %{$tree}) {
-		$output .= $padding.$bullet.'<a class="'.$class.'" href="'.$tree->{$pageId}{url}
-			.'"><span class="selectedMenuItem">'
-			.$tree->{$pageId}{title}.'</span></a>'.$leading;
+		$output .= $padding.$bullet.'<a class="'.$class.'" href="'.$tree->{$pageId}{url}.'">';
+                if ($pageId == $session{page}{pageId}) {
+                       $output .= '<span class="selectedMenuItem">'.$tree->{$pageId}{title}.'</span>';
+                } else {
+                       $output .= $tree->{$pageId}{title};
+                }
+		$output .= '</a>'.$leading;
 		$output .= drawVertical($tree->{$pageId}{sub}, $bullet, $class, $spacing, ($indent+1));
 	}
         return $output;