appdb/include menu.php

[WineHQ]

ChangeSet ID:	31064
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	[e-mail]	2007/04/21 11:54:17

Modified files:
    include        : menu.php

Log message:
    Alexander Nicolaysen Sørnes 
    Include <a> links for menu items. This is necessary for browsers without JavaScript and it
    makes it possible to see the url the link is pointing to

Patch: http://cvs.winehq.org/patch.py?id=31064

Old revision  New revision  Changes     Path
 1.7           1.8           +7 -5       appdb/include/menu.php

Index: appdb/include/menu.php
diff -u -p appdb/include/menu.php:1.7 appdb/include/menu.php:1.8
--- appdb/include/menu.php:1.7	21 Apr 2007 16:54:17 -0000
+++ appdb/include/menu.php	21 Apr 2007 16:54:17 -0000
@@ -33,7 +33,7 @@ class htmlmenu {
     <tr class="topMenu"><td>
       <table width="100%" border="0" cellspacing="0" cellpadding="5">
 ';
-        
+
     }

/* add a table row */
@@ -41,14 +41,16 @@ class htmlmenu {
     {
         if($shUrl)
         {
-            // we have a valid url, make the entire table row clickable and provide some highlighting
-            // for visual feedback
+            // we have a valid url, make the entire table row clickable and provide
+            // some highlighting for visual feedback
             html_tr_highlight_clickable($shUrl, "sideMenu", "#e0e6ff", "#ffffff");
-            echo "<td width='100%' align=$sAlign><span class=MenuItem>&nbsp;<u>$sName</u></span></td>";
+            echo "<td width='100%' align=$sAlign><span class=MenuItem>&nbsp;<u>".
+                    "<a href="$shUrl">$sName</a></u></span></td>";
             echo "</tr>\n";
         } else
         {
-            echo "  <tr class=sideMenu><td width='100%' align=$sAlign><span class=menuItem>&nbsp;$sName</span></td></tr>\n";
+            echo "  <tr class=sideMenu><td width='100%' align=$sAlign><span ".
+                    "class=menuItem>&nbsp;$sName</span></td></tr>\n";
         }
     }