Merge pull request #855 from ExolonDX/branch_01

Tidy up the "Helper Icons"

1 files changed, 11 insertions, 7 deletions

diff --git a/usr/local/www/widgets/widgets/services_status.widget.php b/usr/local/www/widgets/widgets/services_status.widget.php
index 9ae8696..dfe172b 100644
--- a/usr/local/www/widgets/widgets/services_status.widget.php
+++ b/usr/local/www/widgets/widgets/services_status.widget.php
@@ -73,15 +73,19 @@ if (count($services) > 0) {
 		if (empty($service['description']))
 			$service['description'] = get_pkg_descr($service['name']);
 		$service_desc = explode(".",$service['description']);
-		echo '<tr><td class="listlr">' . $service['name'] . '</td>' . "\n";
-		echo '<td class="listr">' . $service_desc[0] . '</td>' . "\n";
-		echo get_service_status_icon($service, false, true);
-		echo '<td valign="middle" class="list nowrap">';
-		echo get_service_control_links($service);
-		echo "</td></tr>\n";
+		echo "<tr><td class=\"listlr\">" . $service['name'] . "</td>\n";
+		echo "<td class=\"listr\">" . $service_desc[0] . "</td>\n";
+		// if service is running then listr else listbg
+		$bgclass = null;
+		if (get_service_status($service))
+			$bgclass = "listr";
+		else
+			$bgclass = "listbg";
+		echo "<td class=\"" . $bgclass . "\" align=\"center\">" . get_service_status_icon($service, true, true) . "</td>\n";
+		echo "<td valign=\"middle\" class=\"list nowrap\">" . get_service_control_links($service) . "</td></tr>\n";
 	}
 } else {
-	echo "<tr><td colspan=\"3\" align=\"center\">" . gettext("No services found") . ".</td></tr>\n";
+	echo "<tr><td colspan=\"3\" align=\"center\">" . gettext("No services found") . " . </td></tr>\n";
 }
 ?>
 </table>