From 3187bac6d3c191a39643a421ca340699222b532e Mon Sep 17 00:00:00 2001 From: Rohit Yadav Date: Wed, 21 Oct 2015 15:24:13 +0530 Subject: [PATCH] metrics: don't sort quick-view column, and bind only one sorting method Signed-off-by: Rohit Yadav --- ui/scripts/ui/widgets/dataTable.js | 9 ++++++--- ui/scripts/ui/widgets/listView.js | 2 +- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/ui/scripts/ui/widgets/dataTable.js b/ui/scripts/ui/widgets/dataTable.js index 24ea7f025e5..f27471d1d39 100644 --- a/ui/scripts/ui/widgets/dataTable.js +++ b/ui/scripts/ui/widgets/dataTable.js @@ -248,9 +248,12 @@ $table.find('tbody').closest('table').addClass('body'); } - $table.find('th:not(:has(input)):not(.collapsible-column)').bind('mousemove mouseout', hoverResizableEvent); - $table.find('th:not(:has(input)):not(.collapsible-column)').bind('mousedown mousemove mouseup mouseout', resizeDragEvent); - $table.find('th:not(:has(input)):not(.collapsible-column)').bind('click', function(event) { + if (!$table.hasClass('horizontal-overflow')) { + $table.find('th:not(:has(input))').bind('mousemove mouseout', hoverResizableEvent); + $table.find('th:not(:has(input))').bind('mousedown mousemove mouseup mouseout', resizeDragEvent); + } + + $table.find('thead tr:last th:not(:has(input)):not(.collapsible-column):not(.quick-view)').unbind('click').bind('click', function(event) { if ($(this).hasClass('resizable')) { return false; } diff --git a/ui/scripts/ui/widgets/listView.js b/ui/scripts/ui/widgets/listView.js index 51ed3be3a60..d7a3bac103a 100644 --- a/ui/scripts/ui/widgets/listView.js +++ b/ui/scripts/ui/widgets/listView.js @@ -1488,8 +1488,8 @@ .appendTo($tr); $quickView.mouseover( // Show quick view - function() { + var $quickView = $(this); var $quickViewTooltip = $('
').addClass('quick-view-tooltip hovered-elem'); var $tr = $quickView.closest('tr'); var $listView = $tr.closest('.list-view');