From c7eca9bda2dacd2bea26e5b1df99b5c28582896d Mon Sep 17 00:00:00 2001
From: Germs2004 <5519340+Germs2004@users.noreply.github.com>
Date: Sun, 27 Mar 2022 16:16:14 -0400
Subject: [PATCH] Remove onclick from non-link table cells

This removes the onclick attribute from the table cells on the index page.  The first cell of each row already has an "a" link which takes the user to the same place and is more intuitive since hovering over that "a" link changes the cursor to a "pointer."  The other cells still showed a "text" cursor which implied the elements were not clickable, though they were.  Removing the onclick from those other cells also allows the user to select their text for copy/paste purposes without getting redirected to another page unexpectedly.
---
 src/templates/default/module/server/status/index.tpl.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/templates/default/module/server/status/index.tpl.html b/src/templates/default/module/server/status/index.tpl.html
index 6102ef64..b8c8cf06 100644
--- a/src/templates/default/module/server/status/index.tpl.html
+++ b/src/templates/default/module/server/status/index.tpl.html
@@ -65,7 +65,7 @@
 				</thead>
 				<tbody>
 					{% for server in servers_offline %}
-						<tr class="bg-danger text-white" onclick="window.location.href='{{ server.url_view|raw }}'">
+						<tr class="bg-danger text-white">
 							<th class="full"><a href="{{ server.url_view|raw }}" class="text-white">{{ server.label }}<span class="sr-only"> ({{ label_offline }})</span></a></th>
 							<td>{{ server.last_online_nice }}</td>
 							<td>{{ server.last_checked_nice }}</td>
@@ -73,7 +73,7 @@
 						</tr>
 					{% endfor %}
 					{% for server in servers_warning %}
-						<tr class="bg-warning text-white" onclick="window.location.href='{{ server.url_view|raw }}'">
+						<tr class="bg-warning text-white">
 							<th class="full"><a href="{{ server.url_view|raw }}" class="text-white">{{ server.label }}<span class="sr-only"> ({{ label_warning }})</span></a></th>
 							<td>{{ server.last_online_nice }}</td>
 							<td>{{ server.last_checked_nice }}</td>
@@ -93,7 +93,7 @@
 				</thead>
 				<tbody>
 					{% for server in servers_online %}
-						<tr class="bg-success text-white" onclick="window.location.href='{{ server.url_view|raw }}'">
+						<tr class="bg-success text-white">
 							<th class="full"><a href="{{ server.url_view|raw }}" class="text-white">{{ server.label }}<span class="sr-only"> ({{ label_online }})</span></a></th>
 							<td>{{ server.last_online_nice }}</td>
 							<td>{{ server.last_offline_nice }} {{ server.last_offline_duration_nice }}</td>