From c2a27ca4693a8327a0c6e990cf376512398cc14e Mon Sep 17 00:00:00 2001
From: Karl Van Iseghem <karl.vaniseghem@icloud.com>
Date: Tue, 30 May 2017 20:17:50 +0200
Subject: [PATCH] Removed inline mouseover/mouseout to toggle sort icon.

Signed-off-by: Karl Van Iseghem <karl.vaniseghem@icloud.com>

Removed inline mouseover/mouseout to toggle sort icon.

Signed-off-by: Karl Van Iseghem <karl.vaniseghem@icloud.com>
---
 js/server_status_processes.js             | 10 ++++++++++
 libraries/server_status_processes.lib.php |  7 +------
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/js/server_status_processes.js b/js/server_status_processes.js
index 87b14f0222..646de2fe5e 100644
--- a/js/server_status_processes.js
+++ b/js/server_status_processes.js
@@ -173,6 +173,14 @@ AJAX.registerOnload('server_status_processes.js', function() {
     $('#tableprocesslist').on('click', 'thead a', function() {
         processList.refreshUrl = $(this).attr('href');
     });
+
+    // Bind event handlers for toggling sort icons
+    $('#tableprocesslist').on('mouseover', 'thead a', function() {
+        $(this).find('.soimg').toggle();
+    });
+    $('#tableprocesslist').on('mouseout', 'thead a', function() {
+        $(this).find('.soimg').toggle();
+    });
 });
 
 /**
@@ -183,6 +191,8 @@ AJAX.registerTeardown('server_status_processes.js', function() {
     $('a#toggleRefresh').off('click');
     $('#id_refreshRate').off('change');
     $('#tableprocesslist').off('click', 'thead a');
+    $('#tableprocesslist').off('mouseover', 'thead a');
+    $('#tableprocesslist').off('mouseout', 'thead a');
     // stop refreshing further
     processList.abortRefresh();
 });
diff --git a/libraries/server_status_processes.lib.php b/libraries/server_status_processes.lib.php
index 75a1574e8a..c379b61f22 100644
--- a/libraries/server_status_processes.lib.php
+++ b/libraries/server_status_processes.lib.php
@@ -144,12 +144,7 @@ function PMA_getHtmlForServerProcesslist()
 
         $retval .= '<th>';
         $columnUrl = URL::getCommon($column);
-        $retval .= '<a href="server_status_processes.php' . $columnUrl . '" ';
-        if ($is_sorted) {
-            $retval .= 'onmouseout="$(\'.soimg\').toggle()" '
-                . 'onmouseover="$(\'.soimg\').toggle()"';
-        }
-        $retval .= '>';
+        $retval .= '<a href="server_status_processes.php' . $columnUrl . '">';
 
         $retval .= $column['column_name'];