From 9c07440e2c57b034344a41ad544335e2243d1152 Mon Sep 17 00:00:00 2001
From: Colin Fleming <cj_fleming@sky.com>
Date: Tue, 5 Mar 2013 16:32:10 +0000
Subject: Tidy up widgets XHTML

Close BR tag
Add CDATA sections to scripts
HTML width paramter must be a whole number, not a decimal.
---
 usr/local/www/widgets/widgets/system_information.widget.php | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

(limited to 'usr')

diff --git a/usr/local/www/widgets/widgets/system_information.widget.php b/usr/local/www/widgets/widgets/system_information.widget.php
index 02e1439..403b24c 100644
--- a/usr/local/www/widgets/widgets/system_information.widget.php
+++ b/usr/local/www/widgets/widgets/system_information.widget.php
@@ -83,8 +83,8 @@ if($_REQUEST['getupdatestatus']) {
 $curcfg = $config['system']['firmware'];
 
 ?>
-</script>
-<table width="100%" border="0" cellspacing="0" cellpadding="0">
+
+<table width="100%" border="0" cellspacing="0" cellpadding="0" summary="system information">
 	<tbody>
 		<tr>
 			<td width="25%" class="vncellt">Name</td>
@@ -98,7 +98,7 @@ $curcfg = $config['system']['firmware'];
 				<br />
 				built on <?php readfile("/etc/version.buildtime"); ?>
 		<br />
-		<div name="uname" id="uname"><a href="#" onClick='swapuname(); return false;'><?php echo php_uname("s") . " " . php_uname("r"); ?></a></div>
+		<div id="uname"><a href="#" onclick='swapuname(); return false;'><?php echo php_uname("s") . " " . php_uname("r"); ?></a></div>
 		<?php if(!isset($config['system']['firmware']['disablecheck'])): ?>
 		<div id='updatestatus'><br/><?php echo gettext("Obtaining update status"); ?> ...</div>
 		<?php endif; ?>
@@ -168,7 +168,7 @@ $curcfg = $config['system']['firmware'];
 					<?php
 						$dns_servers = get_dns_servers();
 						foreach($dns_servers as $dns) {
-							echo "{$dns}<br>";
+							echo "{$dns}<br/>";
 						}
 					?>
 			</td>
@@ -202,7 +202,7 @@ $curcfg = $config['system']['firmware'];
                         <td width="25%" class="vncellt">Temperature</td>
 			<td width="75%" class="listr">
 				<?php $TempMeter = $temp = get_temp(); ?>
-				<img src="./themes/<?= $g['theme']; ?>/images/misc/bar_left.gif" height="15" width="4" border="0" align="middle" alt="left bar" /><img src="./themes/<?= $g['theme']; ?>/images/misc/bar_blue.gif" height="15" name="tempwidtha" id="tempwidtha" width="<?= $TempMeter; ?>" border="0" align="middle" alt="red bar" /><img src="./themes/<?= $g['theme']; ?>/images/misc/bar_gray.gif" height="15" name="tempwidthb" id="tempwidthb" width="<?= (100 - $TempMeter); ?>" border="0" align="middle" alt="gray bar" /><img src="./themes/<?= $g['theme']; ?>/images/misc/bar_right.gif" height="15" width="5" border="0" align="middle" alt="right bar" />
+				<img src="./themes/<?= $g['theme']; ?>/images/misc/bar_left.gif" height="15" width="4" border="0" align="middle" alt="left bar" /><img src="./themes/<?= $g['theme']; ?>/images/misc/bar_blue.gif" height="15" name="tempwidtha" id="tempwidtha" width="<?= round($TempMeter); ?>" border="0" align="middle" alt="red bar" /><img src="./themes/<?= $g['theme']; ?>/images/misc/bar_gray.gif" height="15" name="tempwidthb" id="tempwidthb" width="<?= (100 - $TempMeter); ?>" border="0" align="middle" alt="gray bar" /><img src="./themes/<?= $g['theme']; ?>/images/misc/bar_right.gif" height="15" width="5" border="0" align="middle" alt="right bar" />
 				&nbsp;
 				<span id="tempmeter"><?= $temp."&#176;C"; ?></span>
 			</td>
@@ -255,6 +255,7 @@ $curcfg = $config['system']['firmware'];
 	</tbody>
 </table>
 <script type="text/javascript">
+//<![CDATA[
 	function swapuname() {
 		jQuery('#uname').html("<?php echo php_uname("a"); ?>");
 	}
@@ -278,4 +279,5 @@ $curcfg = $config['system']['firmware'];
 	}
 	setTimeout('getstatus()', 4000);
 	<?php endif; ?>
+//]]>
 </script>
-- 
cgit v1.1