appdb/include menu.php

WineHQ wineowner at wine.codeweavers.com
Sat Apr 21 11:54:17 CDT 2007


ChangeSet ID:	31064
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/04/21 11:54:17

Modified files:
	include        : menu.php 

Log message:
	Alexander Nicolaysen Sørnes <alex at thehandofagony.com>
	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";
         }
     }
 



More information about the wine-cvs mailing list