From 3bc38d3cf2ef2374d77ed2d69b08b0b398c095f1 Mon Sep 17 00:00:00 2001
From: Nate <github@najo.com>
Date: Sat, 2 Nov 2019 20:13:40 -0600
Subject: [PATCH] Active Conferences: Fix list row click.

---
 .../conferences_active_inc.php                | 48 +++++++++++--------
 1 file changed, 29 insertions(+), 19 deletions(-)

diff --git a/app/conferences_active/conferences_active_inc.php b/app/conferences_active/conferences_active_inc.php
index 527715fb7e..8f7e9ebb49 100644
--- a/app/conferences_active/conferences_active_inc.php
+++ b/app/conferences_active/conferences_active_inc.php
@@ -71,13 +71,16 @@
 		$row_style["0"] = "row_style0";
 		$row_style["1"] = "row_style1";
 
-		echo "<table class='tr_hover' width='100%' border='0' cellpadding='0' cellspacing='0'>\n";
-		echo "<tr>\n";
-		echo "<th>".$text['label-name']."</th>\n";
-		echo "<th>".$text['label-participant-pin']."</th>\n";
-		echo "<th>".$text['label-member-count']."</th>\n";
-		echo "<th>&nbsp;</th>\n";
+		echo "<table class='list'>\n";
+		echo "<tr class='list-header'>\n";
+		echo "	<th>".$text['label-name']."</th>\n";
+		echo "	<th>".$text['label-participant-pin']."</th>\n";
+		echo "	<th>".$text['label-member-count']."</th>\n";
+		if (permission_exists('conference_interactive_view')) {
+			echo "	<td class='action-button'>&nbsp;</td>\n";
+		}
 		echo "</tr>\n";
+		$x = 0;
 		foreach ($xml->conference as $row) {
 			//set the variables
 				$name = $row['name'];
@@ -126,24 +129,31 @@
 					}
 
 					if (permission_exists('conference_interactive_view')) {
-						$td_onclick = "onclick=\"document.location.href='conference_interactive.php?c=".urlencode($conference_name)."'\"";
+						$list_row_url = 'conference_interactive.php?c='.urlencode($conference_uuid);
+						$list_row_onclick = "onclick=\"document.location.href='".$list_row_url."';\"";
+					}
+					echo "<tr class='list-row' href='".$list_row_url."'>\n";
+					echo "	<td ".$list_row_onclick.">";
+					if (permission_exists('conference_interactive_view')) {
+						echo "	<a href='".$list_row_url."'>".escape($conference_name)."</a>";
+					}
+					else {
+						echo escape($conference_name);
+					}
+					echo "	</td>\n";
+					echo "	<td ".$list_row_onclick.">".escape($participant_pin)."</td>\n";
+					echo "	<td ".$list_row_onclick.">".escape($member_count)."</td>\n";
+					if (permission_exists('conference_interactive_view')) {
+						echo "	<td class='action-button'>";
+						echo button::create(['type'=>'button','title'=>$text['button-view'],'icon'=>$_SESSION['theme']['button_icon_view'],'link'=>$list_row_url]);
+						echo "	</td>\n";
 					}
-					echo "<tr>\n";
-					echo "<td valign='top' class='".$row_style[$c]."' ".$td_onclick.">";
-					echo (permission_exists('conference_interactive_view')) ? "<a href='conference_interactive.php?c=".urlencode($conference_uuid)."'>".escape($conference_name)."</a>" : escape($conference_name);
-					echo "</td>\n";
-					echo "<td valign='top' class='".$row_style[$c]."' ".$td_onclick.">".escape($participant_pin)."</td>\n";
-					echo "<td valign='top' class='".$row_style[$c]."' ".$td_onclick.">".escape($member_count)."</td>\n";
-					echo "<td valign='top' class='".$row_style[$c]."' ".$td_onclick.">";
-					echo (permission_exists('conference_interactive_view')) ? "<a href='conference_interactive.php?c=".urlencode($conference_uuid)."'>".$text['button-view']."</a>" : "&nbsp;";
-					echo "</td>\n";
 					echo "</tr>\n";
-
-					if ($c==0) { $c=1; } else { $c=0; }
+					$x++;
 				}
 		}
 		echo "</table>\n";
 		echo "<br /><br />";
 	}
 
-?>
+?>
\ No newline at end of file