From 828b1e0a09f8220c04a750a9b9d680cb606fdbbc Mon Sep 17 00:00:00 2001
From: diosmosis <benakamoorthi@fastmail.fm>
Date: Tue, 22 Oct 2013 23:43:29 -0400
Subject: [PATCH] Show column as sorted, even if sort is disabled and do not
 show pointer cursor if sort is not enabled.

---
 plugins/CoreHome/javascripts/dataTable.js     | 32 +++++++++----------
 .../CoreHome/templates/_dataTableHead.twig    |  2 +-
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/plugins/CoreHome/javascripts/dataTable.js b/plugins/CoreHome/javascripts/dataTable.js
index e570bc976c..adc230a3a7 100644
--- a/plugins/CoreHome/javascripts/dataTable.js
+++ b/plugins/CoreHome/javascripts/dataTable.js
@@ -399,23 +399,23 @@ $.extend(DataTable.prototype, UIControl.prototype, {
                     self.onClickSort(this);
                 }
             );
+        }
 
-            if (self.param.filter_sort_column != '') {
-                // are we in a subdatatable?
-                var currentIsSubDataTable = $(domElem).parent().hasClass('cellSubDataTable');
-                var imageSortSrc = getSortImageSrc();
-                var imageSortWidth = 16;
-                var imageSortHeight = 16;
-
-                var sortOrder = self.param.filter_sort_order;
-                var ImageSortClass = sortOrder.charAt(0).toUpperCase() + sortOrder.substr(1);
-
-                // we change the style of the column currently used as sort column
-                // adding an image and the class columnSorted to the TD
-                $(".sortable#" + self.param.filter_sort_column + ' #thDIV', domElem).parent()
-                    .addClass('columnSorted')
-                    .prepend('<div class="sortIconContainer sortIconContainer' + ImageSortClass + '"><img class="sortIcon" width="' + imageSortWidth + '" height="' + imageSortHeight + '" src="' + imageSortSrc + '" /></div>');
-            }
+        if (self.param.filter_sort_column != '') {
+            // are we in a subdatatable?
+            var currentIsSubDataTable = $(domElem).parent().hasClass('cellSubDataTable');
+            var imageSortSrc = getSortImageSrc();
+            var imageSortWidth = 16;
+            var imageSortHeight = 16;
+
+            var sortOrder = self.param.filter_sort_order;
+            var ImageSortClass = sortOrder.charAt(0).toUpperCase() + sortOrder.substr(1);
+
+            // we change the style of the column currently used as sort column
+            // adding an image and the class columnSorted to the TD
+            $("th#" + self.param.filter_sort_column + ' #thDIV', domElem).parent()
+                .addClass('columnSorted')
+                .prepend('<div class="sortIconContainer sortIconContainer' + ImageSortClass + '"><img class="sortIcon" width="' + imageSortWidth + '" height="' + imageSortHeight + '" src="' + imageSortSrc + '" /></div>');
         }
     },
 
diff --git a/plugins/CoreHome/templates/_dataTableHead.twig b/plugins/CoreHome/templates/_dataTableHead.twig
index 75cb111531..43b87a0e69 100644
--- a/plugins/CoreHome/templates/_dataTableHead.twig
+++ b/plugins/CoreHome/templates/_dataTableHead.twig
@@ -1,7 +1,7 @@
 <thead>
    <tr>
        {% for column in properties.columns_to_display %}
-           <th class="sortable {% if loop.first %}first{% elseif loop.last %}last{% endif %}" id="{{ column }}">
+           <th class="{% if properties.enable_sort %}sortable{% endif %} {% if loop.first %}first{% elseif loop.last %}last{% endif %}" id="{{ column }}">
                {% if properties.metrics_documentation[column]|default is not empty %}
                    <div class="columnDocumentation">
                        <div class="columnDocumentationTitle">
-- 
GitLab